tools/metrics/histograms/histograms.xml

Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index f6c9f8713876677091c92fe192141960123c7a80..19ab781c94e23485d39934d60f7117c6c1cb190f 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -62702,6 +62702,22 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   <summary>Track how a profile gets signed out.</summary>
 </histogram>
 
+<histogram name="Signin.TokenTable.GetAllTokensSqlStatement"

[Alexei Svitkine (slow), 2017/02/06 15:35:51]

Nit: Make the histogram name refer to what it's actually logging - i.e. the name
doesn't imply that it's logging whether it's valid...

[msarda, 2017/02/06 16:59:27]

Done.

+    enum="BooleanValid">
+  <owner>msarda@chromium.org</owner>
+  <summary>
+    Counts how many times the SQL statements to get all tokens is valid.
+  </summary>
+</histogram>
+
+<histogram name="Signin.TokenTable.ReadTokenFromDB"

[Alexei Svitkine (slow), 2017/02/06 15:35:51]

Nit: Same comment as above. e.g. maybe ReadTokenFromDB.Result?

[msarda, 2017/02/06 16:59:27]

Done.

+    enum="SigninTokenTableReadTokenFromDB">
+  <owner>msarda@chromium.org</owner>
+  <summary>
+    Counts the results of loading a refresh token from the token database.
+  </summary>
+</histogram>
+
 <histogram name="Signin.UberTokenFailure" enum="GoogleServiceAuthError">
   <owner>mlerman@chromium.org</owner>
   <summary>
@@ -106003,6 +106019,12 @@ value.
   <int value="12" label="Unknown"/>
 </enum>
 
+<enum name="SigninTokenTableReadTokenFromDB" type="int">
+  <int value="0" label="Success"/>
+  <int value="1" label="Decrypt failed"/>
+  <int value="2" label="Read DB failed (bad entry)"/>
+</enum>
+
 <enum name="SigninXDevicePromoEligibility" type="int">
   <int value="0" label="Eligible">The user is eligible for the promo.</int>
   <int value="1" label="Opted Out">