Commit 7cf11376 authored by ttuttle@chromium.org's avatar ttuttle@chromium.org

Domain Reliability: Return http.error for HTTP errors

Since we're already including the HTTP response code as a field in the report,
don't bother including the specific error code in the status as well.

BUG=

Review URL: https://codereview.chromium.org/398173003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283807 0039d316-1c4b-4281-b951-d872f2087c98
parent 6cb2a7fa
...@@ -7,7 +7,6 @@ ...@@ -7,7 +7,6 @@
#include "base/callback.h" #include "base/callback.h"
#include "base/logging.h" #include "base/logging.h"
#include "base/memory/weak_ptr.h" #include "base/memory/weak_ptr.h"
#include "base/strings/stringprintf.h"
#include "base/time/time.h" #include "base/time/time.h"
#include "base/timer/timer.h" #include "base/timer/timer.h"
#include "net/base/net_errors.h" #include "net/base/net_errors.h"
...@@ -84,7 +83,7 @@ bool GetDomainReliabilityBeaconStatus( ...@@ -84,7 +83,7 @@ bool GetDomainReliabilityBeaconStatus(
std::string* beacon_status_out) { std::string* beacon_status_out) {
if (net_error == net::OK) { if (net_error == net::OK) {
if (http_response_code >= 400 && http_response_code < 600) if (http_response_code >= 400 && http_response_code < 600)
*beacon_status_out = base::StringPrintf("http.%d", http_response_code); *beacon_status_out = "http.error";
else else
*beacon_status_out = "ok"; *beacon_status_out = "ok";
return true; return true;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment