[NTP] Fixed usage of NewTabPage.ActionAndroid histogram

tools/metrics/actions/actions.xml

Index: tools/metrics/actions/actions.xml
diff --git a/tools/metrics/actions/actions.xml b/tools/metrics/actions/actions.xml
index 0961d3827d7deb6823d0f025780644702c356092..b4fa25e9181cd2e4163c30cb115543ce1e6f55e3 100644
--- a/tools/metrics/actions/actions.xml
+++ b/tools/metrics/actions/actions.xml
@@ -9199,6 +9199,13 @@ should be able to be added at any place in this file.
   </description>
 </action>
 
+<action name="MobileBookmarkOpen">
+  <owner>ianwen@chromium.org</owner>
+  <description>
+    Action indicating the user has opened a bookmark from the bookmark manager.

[Mark P, 2016/12/21 18:13:13]

optional nit (general advice; I see it doesn't agree with some other mobile
bookmark action descriptions though is a relatively common practice throughout
this file): omit unnecessary words/phrases, e.g., drop "Action indicating" or
maybe even "Action indicating the user has"

[finkm, 2016/12/22 09:45:44]

Done

+  </description>
+</action>
+
 <action name="MobileBreakpadUploadAttempt" not_user_triggered="true">
   <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
   <description>Please enter the description of this user action.</description>
@@ -9400,6 +9407,14 @@ should be able to be added at any place in this file.
   <obsolete>Deprecated as of 12/2015</obsolete>
 </action>
 
+<action name="MobileForeignSessionOpen">
+  <owner>finkm@chromium.org</owner>
+  <description>
+    Action indicating the user has opened a foreign session from the recent tabs

[Mark P, 2016/12/21 18:13:13]

nit: "foreign session" is generally only a term that member of the team know. 
The broader chromium community thinks of these as tabs from other devices.  For
better discoverability and understandability, I'd prefer you choose a better
name such as MobileOpenTabFromOtherDevice or MobileTabFromOtherDeviceOpen.

Even if you decide not to change the name, the description should mention "other
devices" because I can easily imagine people saying, I hope we log this, and
going to actions.xml and searching and hoping to find something.

[finkm, 2016/12/22 09:45:44]

Done

+    manager.
+  </description>
+</action>
+
 <action name="MobileFre.SignInShown">
   <owner>aruslan@chromium.org</owner>
   <description>
@@ -9691,6 +9706,7 @@ should be able to be added at any place in this file.
   <description>
     Action indicating the user has opened a bookmark from the bookmark manager.
   </description>
+  <obsolete>Deprecated as of 11/2016</obsolete>

[Mark P, 2016/12/21 18:13:13]

Please check to see whether this action is emitted on iOS.

[finkm, 2016/12/22 09:45:44]

You're right. It's still used in iOS. I removed the obsolete tag.

 </action>
 
 <action name="MobileNTPForeignSession">
@@ -9920,6 +9936,14 @@ should be able to be added at any place in this file.
   <description>Please enter the description of this user action.</description>
 </action>
 
+<action name="MobileRecentlyClosedTabOpen">
+  <owner>finkm@chromium.org</owner>
+  <description>
+    Action indicating the user has opened a recently closed tab from the recent
+    tabs manager.
+  </description>
+</action>
+
 <action name="MobileRendererCrashed">
   <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
   <description>Please enter the description of this user action.</description>