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 60509425f675ded34432e7dcbd20cbf98cb217e6..ad1c63e73b028c06cf703681d632937b5f236aa1 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -4117,6 +4117,36 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="ContextMenu.Image" enum="ContextMenuAction">

[Alexei Svitkine (slow), 2015/08/04 21:29:13]

Since these are Android specific, perhaps they should be named
Android.ContextMenu.*? Or are you planning to add these for other platforms as
well?

(One reason I'm asking is that we try to avoid adding many new top-level
prefixes, since this clutters the top-level list in some of our dashboard UIs.
So finding existing top level prefixes to nest under is preferable.)

+  <owner>newt@chromium.org</owner>
+  <summary>
+    The option that the user selected from the context menu for an image.
+  </summary>
+</histogram>
+
+<histogram name="ContextMenu.ImageLink" enum="ContextMenuAction">
+  <owner>newt@chromium.org</owner>
+  <summary>
+    The option that the user selected from the context menu for an image that is
+    also a link.
+  </summary>
+</histogram>
+
+<histogram name="ContextMenu.Link" enum="ContextMenuAction">
+  <owner>newt@chromium.org</owner>
+  <summary>
+    The option that the user selected from the context menu for a regular link
+    (not an image or video that's also a link).
+  </summary>
+</histogram>
+
+<histogram name="ContextMenu.Video" enum="ContextMenuAction">

[Alexei Svitkine (slow), 2015/08/04 21:29:13]

Since you have a number of suffixes that split the metrics depending on what
you're context menu-ing on, I suggest using the histogram_suffixes feature of
the XML format to make this more concise. See comment at the top of the file for
an example.

[newt (away), 2015/08/12 23:43:20]

Done.

+  <owner>newt@chromium.org</owner>
+  <summary>
+    The option that the user selected from the context menu for a video.
+  </summary>
+</histogram>
+
 <histogram name="Cookie.BackingStoreUpdateResults" enum="BackingStoreResults">
   <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
   <summary>
@@ -52884,6 +52914,25 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   <int value="11" label="CONTEXT_LOST_MAKECURRENT_FAILED"/>
 </enum>
 
+<enum name="ContextMenuAction" type="int">
+  <summary>The item selected from a context menu</summary>
+  <int value="0" label="Open in new tab"/>
+  <int value="1" label="Open in incognito tab"/>
+  <int value="2" label="Copy link address"/>
+  <int value="3" label="Copy email address"/>
+  <int value="4" label="Copy link text"/>
+  <int value="5" label="Save link"/>
+  <int value="6" label="Save image"/>
+  <int value="7" label="Open image"/>
+  <int value="8" label="Open image in new tab"/>
+  <int value="9" label="Copy image"/>
+  <int value="10" label="Copy image URL"/>
+  <int value="11" label="Search by image"/>
+  <int value="12" label="Load images"/>
+  <int value="13" label="Load original image"/>
+  <int value="14" label="Save video"/>
+</enum>
+
 <enum name="CookieDeletionCause" type="int">
   <summary>Reason why a cookie was removed from the cookie store</summary>
   <int value="0" label="explicit">