Commit e7ec7c64 authored by ramankk@chromium.org's avatar ramankk@chromium.org

rAc: Wallet: UMA for wallet response codes.

BUG=261814

Review URL: https://chromiumcodereview.appspot.com/23461033

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@221713 0039d316-1c4b-4281-b951-d872f2087c98
parent a3b936d9
...@@ -595,6 +595,8 @@ void WalletClient::OnURLFetchComplete( ...@@ -595,6 +595,8 @@ 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,6 +6,7 @@ ...@@ -6,6 +6,7 @@
#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"
...@@ -386,6 +387,10 @@ void AutofillMetrics::LogWalletRequiredActionMetric( ...@@ -386,6 +387,10 @@ 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,6 +389,9 @@ class AutofillMetrics { ...@@ -389,6 +389,9 @@ 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,6 +17436,10 @@ other types of suffix sets. ...@@ -17436,6 +17436,10 @@ 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
...@@ -21172,6 +21176,49 @@ other types of suffix sets. ...@@ -21172,6 +21176,49 @@ 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