Commit 61ea1362 authored by fgorski@chromium.org's avatar fgorski@chromium.org

Revert 221713 "rAc: Wallet: UMA for wallet response codes."

> rAc: Wallet: UMA for wallet response codes.
> 
> BUG=261814
> 
> Review URL: https://chromiumcodereview.appspot.com/23461033

TBR=ramankk@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@221718 0039d316-1c4b-4281-b951-d872f2087c98
parent a9b875bf
...@@ -595,8 +595,6 @@ void WalletClient::OnURLFetchComplete( ...@@ -595,8 +595,6 @@ void WalletClient::OnURLFetchComplete(
scoped_ptr<base::DictionaryValue> response_dict; scoped_ptr<base::DictionaryValue> response_dict;
int response_code = source->GetResponseCode(); int response_code = source->GetResponseCode();
delegate_->GetMetricLogger().LogWalletResponseCode(response_code);
switch (response_code) { switch (response_code) {
// HTTP_BAD_REQUEST means the arguments are invalid. No point retrying. // HTTP_BAD_REQUEST means the arguments are invalid. No point retrying.
case net::HTTP_BAD_REQUEST: { case net::HTTP_BAD_REQUEST: {
......
...@@ -6,7 +6,6 @@ ...@@ -6,7 +6,6 @@
#include "base/logging.h" #include "base/logging.h"
#include "base/metrics/histogram.h" #include "base/metrics/histogram.h"
#include "base/metrics/sparse_histogram.h"
#include "base/time/time.h" #include "base/time/time.h"
#include "components/autofill/core/browser/autofill_type.h" #include "components/autofill/core/browser/autofill_type.h"
#include "components/autofill/core/browser/form_structure.h" #include "components/autofill/core/browser/form_structure.h"
...@@ -387,10 +386,6 @@ void AutofillMetrics::LogWalletRequiredActionMetric( ...@@ -387,10 +386,6 @@ void AutofillMetrics::LogWalletRequiredActionMetric(
required_action, NUM_WALLET_REQUIRED_ACTIONS); required_action, NUM_WALLET_REQUIRED_ACTIONS);
} }
void AutofillMetrics::LogWalletResponseCode(int response_code) const {
UMA_HISTOGRAM_SPARSE_SLOWLY("Wallet.ResponseCode", response_code);
}
void AutofillMetrics::LogDeveloperEngagementMetric( void AutofillMetrics::LogDeveloperEngagementMetric(
DeveloperEngagementMetric metric) const { DeveloperEngagementMetric metric) const {
DCHECK_LT(metric, NUM_DEVELOPER_ENGAGEMENT_METRICS); DCHECK_LT(metric, NUM_DEVELOPER_ENGAGEMENT_METRICS);
......
...@@ -389,9 +389,6 @@ class AutofillMetrics { ...@@ -389,9 +389,6 @@ class AutofillMetrics {
virtual void LogWalletRequiredActionMetric( virtual void LogWalletRequiredActionMetric(
WalletRequiredActionMetric required_action) const; WalletRequiredActionMetric required_action) const;
// Logs HTTP response codes recieved by wallet client.
virtual void LogWalletResponseCode(int response_code) const;
// This should be called when a form that has been Autofilled is submitted. // This should be called when a form that has been Autofilled is submitted.
// |duration| should be the time elapsed between form load and submission. // |duration| should be the time elapsed between form load and submission.
virtual void LogFormFillDurationFromLoadWithAutofill( virtual void LogFormFillDurationFromLoadWithAutofill(
......
...@@ -17436,10 +17436,6 @@ other types of suffix sets. ...@@ -17436,10 +17436,6 @@ other types of suffix sets.
</summary> </summary>
</histogram> </histogram>
<histogram name="Wallet.ResponseCode" enum="HttpResponseCode">
<summary>HTTP response codes seen by Wallet client.</summary>
</histogram>
<histogram name="WebCore.Animation.CSSProperties" enum="MappedCSSProperties"> <histogram name="WebCore.Animation.CSSProperties" enum="MappedCSSProperties">
<summary> <summary>
Counts the number of times each CSS property is animated. There is no limit Counts the number of times each CSS property is animated. There is no limit
...@@ -21176,49 +21172,6 @@ other types of suffix sets. ...@@ -21176,49 +21172,6 @@ other types of suffix sets.
<int value="9" label="Corrupt stats response"/> <int value="9" label="Corrupt stats response"/>
</enum> </enum>
<enum name="HttpResponseCode" type="int">
<int value="100" label="Continue"/>
<int value="101" label="Switching Protocols"/>
<int value="200" label="OK"/>
<int value="201" label="Created"/>
<int value="202" label="Accepted"/>
<int value="203" label="Non-Authoritative Information"/>
<int value="204" label="No Content"/>
<int value="205" label="Reset Content"/>
<int value="206" label="Partial Content"/>
<int value="300" label="Multiple Choices"/>
<int value="301" label="Moved Permanently"/>
<int value="302" label="Found"/>
<int value="303" label="See Other"/>
<int value="304" label="Not Modified"/>
<int value="305" label="Use Proxy"/>
<int value="306" label="(Unused)"/>
<int value="307" label="Temporary Redirect"/>
<int value="400" label="Bad Request"/>
<int value="401" label="Unauthorized"/>
<int value="402" label="Payment Required"/>
<int value="403" label="Forbidden"/>
<int value="404" label="Not Found"/>
<int value="405" label="Method Not Allowed"/>
<int value="406" label="Not Acceptable"/>
<int value="407" label="Proxy Authentication Required"/>
<int value="408" label="Request Timeout"/>
<int value="409" label="Conflict"/>
<int value="410" label="Gone"/>
<int value="411" label="Length Required"/>
<int value="412" label="Precondition Failed"/>
<int value="413" label="Request Entity Too Large"/>
<int value="414" label="Request-URI Too Long"/>
<int value="415" label="Unsupported Media Type"/>
<int value="416" label="Requested Range Not Satisfiable"/>
<int value="417" label="Expectation Failed"/>
<int value="500" label="Internal Server Error"/>
<int value="501" label="Not Implemented"/>
<int value="503" label="Service Unavailable"/>
<int value="504" label="Gateway Timeout"/>
<int value="505" label="HTTP Version Not Supported"/>
</enum>
<enum name="HttpSocketType" type="int"> <enum name="HttpSocketType" type="int">
<int value="0" label="UNUSED">newly connected socket</int> <int value="0" label="UNUSED">newly connected socket</int>
<int value="1" label="UNUSED_IDLE"> <int value="1" label="UNUSED_IDLE">
......
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