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

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

Issue 2528303003: [NTP] Fixed usage of NewTabPage.ActionAndroid histogram (Closed)
Patch Set: Rebase 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 1c2afd9a926e50ef5d6d809827c18bb7ead984ad..a3e5ddb84a4ce30e20022e354875877abbbb2f01 100644
--- a/tools/metrics/actions/actions.xml
+++ b/tools/metrics/actions/actions.xml
@@ -9308,6 +9308,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>
@@ -9823,11 +9832,18 @@ 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. Replaced with MobileBookmarkManagerEntryOpened.
+ </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. Replaced with
+ MobileRecentTabManagerTabFromOtherDeviceOpened.
+ </obsolete>
</action>
<action name="MobileNTPMostVisited">
@@ -9838,6 +9854,10 @@ 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. Replaced with
+ MobileRecentTabManagerRecentTabOpened.
+ </obsolete>
</action>
<action name="MobileNTPShown">
@@ -10077,6 +10097,24 @@ should be able to be added at any place in this file.
</description>
</action>
+<action name="MobileRecentTabManagerRecentTabOpened">
+ <owner>finkm@chromium.org</owner>
+ <description>
+ A recently closed tab was opened from the recent tabs manager. Recently
+ 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. Tabs from
+ 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