[NTP] Fixed usage of NewTabPage.ActionAndroid histogram

tools/metrics/actions/actions.xml

 <!--
 Copyright 2014 The Chromium Authors. All rights reserved.
 Use of this source code is governed by a BSD-style license that can be
 found in the LICENSE file.
 -->
 
 <!--
 This file is a comprehensive list of Chrome user actions along with the owner
 and description for each user action.
 
@@ skipping 9181 matching lines @@
 
 <action name="MobileBookmarkManagerPageOpen">
   <owner>ianwen@chromium.org</owner>
   <description>
     Action indictating the bookmark manager is created as a Native Page. The
     porpuse of this action is to tell how many users on phones are able to get
     access to bookmark manager inside of a tab.
   </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>
 </action>
 
 <action name="MobileBreakpadUploadFail" not_user_triggered="true">
   <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
   <description>Please enter the description of the metric.</description>
 </action>
 
@@ skipping 181 matching lines @@
   <description>Please enter the description of this user action.</description>
   <obsolete>Deprecated as of 12/2015</obsolete>
 </action>
 
 <action name="MobileFocusedOmniboxOnNtp">
   <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
   <description>Please enter the description of this user action.</description>
   <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>
     User was presented with the sign-in dialog during the First Run Experience.
     This metric is specific to Android.
   </description>
 </action>
 
 <action name="MobileFreAttemptSignIn">
   <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
@@ skipping 271 matching lines @@
     Android: User navigated back to the NTP after reading an opened (clicked)
     snippet card.
   </description>
 </action>
 
 <action name="MobileNTPBookmark">
   <owner>ianwen@chromium.org</owner>
   <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">
   <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
   <description>Please enter the description of this user action.</description>
 </action>
 
 <action name="MobileNTPMostVisited">
   <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
   <description>Please enter the description of this user action.</description>
@@ skipping 209 matching lines @@
 <action name="MobileReadingListOpen">
   <owner>gambard@chromium.org</owner>
   <description>User opened a ReadingList entry.</description>
 </action>
 
 <action name="MobileReceivedExternalIntent">
   <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
   <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>
 </action>
 
 <action name="MobileSadTabFeedback">
   <owner>jwanda@chromium.org</owner>
   <description>
     The user pressed the Send Feedback button on a tab that is displaying the
     sad view.
@@ skipping 6175 matching lines @@
 <action name="webapps.AddShortcut.Manifest">
   <owner>mlamouri@chromium.org</owner>
   <description>
     Shortcut added to the homescreen with a valid Manifest. This action will be
     recorded in addition to AppShortcut, AppShortcutApple and Bookmark actions
     to keep a trend of whether the webapp also add meta tags.
   </description>
 </action>
 
 </actions>