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 d75f373bd7fa28db7b412d0f95d1bf127c11fed2..15b5de1c330e32a9860340d7594644d50380aaf1 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -600,6 +600,25 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="Ash.Accelerators.LockScreen" enum="DeprecatedAcceleratorUsage">
+  <owner>afakhry@chromium.org</owner>
+  <summary>
+    The lock screen action has two accelerators:   - Ctrl+Shift+L which is
+    deprecated.   - Search+L which is new. This histogram tells us how many
+    users use the old vs the new one.

[Mark P, 2015/06/12 18:01:44]

This second sentence needs to be revise.  It needs to
- explicitly state when an item is emitted to the histogram
- don't state incorrect things.  This histogram does not tell how many users use
the old versus the new one (unless you're referring to the "User Counts"
selection from histogram dashboard, which I doubt; regardless, the histogram
description should be correct regardless of what view / grouping of the data the
user is using.).  It tells us some sort of thing about frequency/counts of use
(which should be made clear simply by fixing the first bullet).

ditto below

[afakhry, 2015/06/12 22:05:54]

You're right. I didn't mean the number of users but rather the number of times
each accelerator type (deprecated or new) is used.

Done.

+  </summary>
+</histogram>
+
+<histogram name="Ash.Accelerators.ShowTaskManager"
+    enum="DeprecatedAcceleratorUsage">
+  <owner>afakhry@chromium.org</owner>
+  <summary>
+    Opening the task manager has two accelerators:   - Shift+Esc which is
+    deprecated.   - Search+Esc which is new. This histogram tells us how many
+    users use the old vs the new one.
+  </summary>
+</histogram>
+
 <histogram name="Ash.ActiveTouchPoints">
   <owner>kuscher@google.com</owner>
   <owner>rbyers@chromium.org</owner>
@@ -51498,6 +51517,11 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   <int value="3" label="Invalid delays">Insufficient amount of data.</int>
 </enum>
 
+<enum name="DeprecatedAcceleratorUsage" type="int">
+  <int value="0" label="Deprecated key accelerator is used"/>
+  <int value="1" label="New key accelerator is used"/>
+</enum>
+
 <enum name="DesktopCaptureCounters" type="int">
   <int value="0" label="Screen capturer created."/>
   <int value="1" label="Window capturer created."/>