UMA for pin unlock success/failure.

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 cc783b848d0b4883dbfe31779a3ded257ce5dda8..4477ddbcaa85449ce01d7899632a607cdfbcf7aa 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -51199,6 +51199,22 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   <summary>How long it takes the swap ack to return after a swap.</summary>
 </histogram>
 
+<histogram name="ScreenLocker.AuthenticationFailure" enum="UnlockFailureType">
+  <owner>sammiequon@chromium.org</owner>
+  <summary>
+    What type of authentication was attempted when the user failed to unlock the
+    lock screen.
+  </summary>
+</histogram>
+
+<histogram name="ScreenLocker.AuthenticationSuccess" enum="UnlockSuccessType">
+  <owner>sammiequon@chromium.org</owner>
+  <summary>
+    What type of authentication was attempted when the user successfully
+    unlocked the lock screen.
+  </summary>
+</histogram>
+
 <histogram name="Sdch3.AdvertisedWithSecureScheme" enum="BooleanHttps">
   <obsolete>
     Experiment complete, histogram gathering code removed.
@@ -95398,6 +95414,16 @@ To add a new entry, add it with any value and run test to compute valid value.
   <int value="3" label="Group not reported and trial was disabled"/>
 </enum>
 
+<enum name="UnlockFailureType" type="int">
+  <int value="0" label="Password"/>
+  <int value="1" label="Pin"/>
+</enum>
+
+<enum name="UnlockSuccessType" type="int">
+  <int value="0" label="Password"/>
+  <int value="1" label="Pin"/>
+</enum>

[Ilya Sherman, 2016/08/18 21:42:20]

nit: Any reason not to combine these two enums into a single one?

[sammiequon, 2016/08/19 16:47:22]

Done.

+
 <enum name="UpdateEngineAttemptResult" type="int">
   <int value="0" label="Update Succeeded"/>
   <int value="1" label="Internal Error"/>