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

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

Issue 2341593003: Add a user action for stop in Android. (Closed)
Patch Set: mpearson@s comment Created 4 years, 3 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
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/actions/actions.xml
diff --git a/tools/metrics/actions/actions.xml b/tools/metrics/actions/actions.xml
index 9afb9af7610658bda04282cf906fa3b62cda3b18..993e5c355a489c134eda177f2b094ce5c6c2d25f 100644
--- a/tools/metrics/actions/actions.xml
+++ b/tools/metrics/actions/actions.xml
@@ -9369,9 +9369,12 @@ should be able to be added at any place in this file.
</action>
<action name="MobileMenuReload">
- <owner>aurimas@chromium.org</owner>
- <description>User pressed 'Reload' in the app menu.</description>
- <obsolete>This menu item was never added to the app menu.</obsolete>
+ <owner>tedchoc@chromium.org</owner>
+ <description>
+ User pressed the reload option in the app menu. This was added in M55.
+ Prior to M55, the menu reload counts were combined with the toolbar counts,
+ which were all reported as MobileToolbarReload.
+ </description>
</action>
<action name="MobileMenuRequestDesktopSite">
@@ -9396,6 +9399,11 @@ should be able to be added at any place in this file.
<description>User opened the app menu.</description>
</action>
+<action name="MobileMenuStop">
+ <owner>tedchoc@chromium.org</owner>
+ <description>User pressed the stop option in the app menu.</description>
+</action>
+
<action name="MobileNTP.Interests.Click">
<owner>knn@chromium.org</owner>
<description>
@@ -9789,8 +9797,12 @@ should be able to be added at any place in this file.
</action>
<action name="MobileToolbarReload">
- <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
- <description>Please enter the description of this user action.</description>
+ <owner>tedchoc@chromium.org</owner>
+ <description>
+ User pressed the reload icon in the toolbar. Pre-M55, MobileToolbarReload
+ accounted for all reload actions. M55+, this metric only refers to reloads
+ that occurred from the toolbar.
+ </description>
</action>
<action name="MobileToolbarShowMenu">
@@ -9808,6 +9820,11 @@ 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="MobileToolbarStop">
+ <owner>tedchoc@chromium.org</owner>
+ <description>User pressed the stop icon in the toolbar.</description>
+</action>
+
<action name="MobileToolbarToggleBookmark">
<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 | « chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698