Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(150)

Unified Diff: tools/metrics/actions/actions.xml

Issue 2528303003: [NTP] Fixed usage of NewTabPage.ActionAndroid histogram (Closed)
Patch Set: Changed order of UMA calls in ChromeTabbedActivity.java Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/metrics/actions/actions.xml
diff --git a/tools/metrics/actions/actions.xml b/tools/metrics/actions/actions.xml
index e2aa4a4417315256e340220cec434defa4d37644..6d8990d7a8009d934042c652193e6c95345b3d27 100644
--- a/tools/metrics/actions/actions.xml
+++ b/tools/metrics/actions/actions.xml
@@ -9182,6 +9182,15 @@ 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="MobileBookmarkManagerEntryOpened">
+ <owner>ianwen@chromium.org</owner>
+ <description>
+ A bookmark was opened from the bookmark manager. Bookmarks opened directly
+ from the NTP will not be counted. Instead those will be counted in the
+ NewTabPage.ContentSuggestions.* histograms.
+ </description>
+</action>
+
<action name="MobileBookmarkManagerOpen">
<owner>ianwen@chromium.org</owner>
<description>
@@ -9691,11 +9700,13 @@ 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 01/2017</obsolete>
</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>
+ <obsolete>Deprecated as of 01/2017</obsolete>
Mark P 2017/01/06 00:04:39 Please add replaced with MobileRecentTabManagerTab
finkm 2017/01/09 10:16:32 3x done!
</action>
<action name="MobileNTPMostVisited">
@@ -9706,6 +9717,7 @@ should be able to be added at any place in this file.
<action name="MobileNTPRecentlyClosed">
<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 01/2017</obsolete>
</action>
<action name="MobileNTPShown">
@@ -9920,6 +9932,24 @@ 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="MobileRecentTabManagerRecentTabOpened">
+ <owner>finkm@chromium.org</owner>
+ <description>
+ A recently closed tab was opened from the recent tabs manager.Recently
Mark P 2017/01/06 00:04:39 nit: space after period.
finkm 2017/01/09 10:16:32 done
+ closed tabs opened directly from the NTP will not be counted. Instead those
+ will be counted in the NewTabPage.ContentSuggestions.* histograms.
+ </description>
+</action>
+
+<action name="MobileRecentTabManagerTabFromOtherDeviceOpened">
+ <owner>finkm@chromium.org</owner>
+ <description>
+ A tab from another device was opened from the recent tabs manager. Tab from
Mark P 2017/01/06 00:04:39 nit: Tab from -> Tabs from
finkm 2017/01/09 10:16:32 done
+ another device opened directly from the NTP will not be counted. Instead
+ those will be counted in the NewTabPage.ContentSuggestions.* histograms.
+ </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>
« no previous file with comments | « ios/chrome/browser/ui/ntp/recent_tabs/recent_tabs_table_view_controller.mm ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698