Add UMA stats to track whitelist download latency.

components/autofill/browser/autofill_metrics.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_AUTOFILL_BROWSER_AUTOFILL_METRICS_H_
 #define COMPONENTS_AUTOFILL_BROWSER_AUTOFILL_METRICS_H_
 
 #include <stddef.h>
 #include <string>
 
@@ skipping 332 matching lines @@
     ACCEPT_TOS,
     UPDATE_EXPIRATION_DATE,
     UPGRADE_MIN_ADDRESS,
     CHOOSE_ANOTHER_INSTRUMENT_OR_ADDRESS,
     VERIFY_CVV,
     INVALID_FORM_FIELD,
     REQUIRE_PHONE_NUMBER,
     NUM_WALLET_REQUIRED_ACTIONS
   };
 
+  // The success or failure of downloading Autocheckout whitelist file.
+  enum WhitelistDownloadStatus {

[ahutter, 2013/04/16 17:56:54]

Probably want to throw Autocheckout on the front of this.

[benquan, 2013/04/17 01:51:30]

Done.

+    // The download was failed.

[ahutter, 2013/04/16 17:56:54]

The download failed.

[Ilya Sherman, 2013/04/17 00:18:01]

nit: This comment is almost verbatim identical to the enum constant name, so I
think you can just drop it.  Ditto for the comment below.

[benquan, 2013/04/17 01:51:30]

Done.

[benquan, 2013/04/17 05:48:18]

Done.

+    WHITELIST_DOWNLOAD_FAILED,

[ahutter, 2013/04/16 17:56:54]

Ditto about AUTOCHECKOUT

[benquan, 2013/04/17 01:51:30]

Done.

+    // The download was succeeded.

[ahutter, 2013/04/16 17:56:54]

The download was successful.

[benquan, 2013/04/17 01:51:30]

Done.

+    WHITELIST_DOWNLOAD_SUCCEEDED,

[ahutter, 2013/04/16 17:56:54]

ditto about AUTOCHECKOUT.

[benquan, 2013/04/17 01:51:30]

Done.

+  };
+
   AutofillMetrics();
   virtual ~AutofillMetrics();
 
   // Logs how the user interacted with the Autocheckout bubble.
   virtual void LogAutocheckoutBubbleMetric(BubbleMetric metric) const;
 
   // Logs the result of an Autocheckout buy flow.
   virtual void LogAutocheckoutBuyFlowMetric(
       AutocheckoutBuyFlowMetric metric) const;
 
@@ skipping 65 matching lines @@
 
   // Logs |required_action| to the required actions histogram for |dialog_type|.
   virtual void LogWalletRequiredActionMetric(
       autofill::DialogType dialog_type,
       WalletRequiredActionMetric required_action) const;
 
   virtual void LogAutocheckoutDuration(
       const base::TimeDelta& duration,
       AutocheckoutCompletionStatus status) const;
 
+  virtual void LogAutocheckoutWhitelistDownloadDuration(

[ahutter, 2013/04/16 17:56:54]

Docs.

[benquan, 2013/04/17 01:51:30]

Done.

+      const base::TimeDelta& duration, WhitelistDownloadStatus status) const;

[Ilya Sherman, 2013/04/17 00:18:01]

nit: One parameter per line, please, if they have to wrap.

[benquan, 2013/04/17 01:51:30]

Done.

+
   // This should be called when a form that has been Autofilled is submitted.
   // |duration| should be the time elapsed between form load and submission.
   virtual void LogFormFillDurationFromLoadWithAutofill(
       const base::TimeDelta& duration) const;
 
   // This should be called when a fillable form that has not been Autofilled is
   // submitted.  |duration| should be the time elapsed between form load and
   // submission.
   virtual void LogFormFillDurationFromLoadWithoutAutofill(
       const base::TimeDelta& duration) const;
@@ skipping 31 matching lines @@
   virtual void LogServerExperimentIdForUpload(
       const std::string& experiment_id) const;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(AutofillMetrics);
 };
 
 }  // namespace autofill
 
 #endif  // COMPONENTS_AUTOFILL_BROWSER_AUTOFILL_METRICS_H_