New encryption/escrow endpoints for Wallet

components/autofill/core/browser/autofill_metrics.h

Index: components/autofill/core/browser/autofill_metrics.h
diff --git a/components/autofill/core/browser/autofill_metrics.h b/components/autofill/core/browser/autofill_metrics.h
index 5fde62c2fbe825d97c874cfd4705d0ed130240ab..b90a0b653fc770127ab1ff572fd0f84bc57c84ef 100644
--- a/components/autofill/core/browser/autofill_metrics.h
+++ b/components/autofill/core/browser/autofill_metrics.h
@@ -287,19 +287,16 @@ class AutofillMetrics {
   };
 
   // For measuring the network request time of various Wallet API calls. See
-  // WalletClient::RequestType.
+  // WalletClient::RequestType. A number of members of this enum have been
+  // deprecated and removed.

[Ilya Sherman, 2013/07/02 21:40:38]

Please keep them in the enum, and pre-pend the entries with "DEPRECATED_".

[Dan Beam, 2013/07/02 21:41:35]

i suggested just removing them, guess this is the way to go -- sorry ahutter@

[Dan Beam, 2013/07/02 21:42:15]

any particular reason for this? just to be less error prone?

[Ilya Sherman, 2013/07/02 21:49:07]

Yep, less error-prone.

[ahutter, 2013/07/02 21:56:54]

Done.

   enum WalletApiCallMetric {
     UNKNOWN_API_CALL,  // Catch all. Should never be used.
     ACCEPT_LEGAL_DOCUMENTS,
     AUTHENTICATE_INSTRUMENT,
     GET_FULL_WALLET,
     GET_WALLET_ITEMS,
-    SAVE_ADDRESS,
-    SAVE_INSTRUMENT,
-    SAVE_INSTRUMENT_AND_ADDRESS,
-    SEND_STATUS,
-    UPDATE_ADDRESS,
-    UPDATE_INSTRUMENT,
+    SEND_STATUS = 8,
+    SAVE_TO_WALLET = 11,
   };
 
   // For measuring the frequency of errors while communicating with the Wallet