Chromium Code Reviews Chromium Code Reviews
Issue 424943002:
Add UMA for app info dialog (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: tools/metrics/histograms/histograms.xml |
| diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml |
| index 0ef82fff9f2756cf671835d0477609ee582d3078..cf2b3bbb28befa1498830144fdb845782115df73 100644 |
| --- a/tools/metrics/histograms/histograms.xml |
| +++ b/tools/metrics/histograms/histograms.xml |
| @@ -418,6 +418,25 @@ Therefore, the affected-histogram name has to have at least one dot in it. |
| </summary> |
| </histogram> |
| +<histogram name="Apps.AppInfoDialogLaunched" enum="AppInfoDialogLaunchSource"> |
|
Alexei Svitkine (slow)
2014/08/06 16:46:44
For these histograms, I suggest naming them Apps.A
sashab
2014/08/11 01:32:06
Done.
|
| + <owner>sashab@chromium.org</owner> |
| + <summary> |
| + The number of times the app info dialog is launched. This is gathered each |
| + time the app info dialog is opened, from each of the places the dialog can |
| + be opened from. |
| + </summary> |
| +</histogram> |
| + |
| +<histogram name="Apps.AppInfoDialogOpenedForLocation" enum="AppLocation"> |
| + <owner>sashab@chromium.org</owner> |
| + <summary>The location of the app that the dialog was opened for.</summary> |
|
Alexei Svitkine (slow)
2014/08/06 16:46:44
Mention that it's logged every time the app info d
sashab
2014/08/11 01:32:06
Done.
|
| +</histogram> |
| + |
| +<histogram name="Apps.AppInfoDialogOpenedForType" enum="ExtensionType"> |
| + <owner>sashab@chromium.org</owner> |
| + <summary>The type of the app that the dialog was opened for.</summary> |
|
Alexei Svitkine (slow)
2014/08/06 16:46:44
Mention that it's logged every time the app info d
sashab
2014/08/11 01:32:06
Done.
|
| +</histogram> |
| + |
| <histogram name="Apps.AppLaunch" enum="AppLaunch"> |
| <owner>Please list the metric's owners. Add more owner tags as needed.</owner> |
| <summary> |
| @@ -35907,6 +35926,10 @@ Therefore, the affected-histogram name has to have at least one dot in it. |
| <int value="8" label="Cancelled"/> |
| </enum> |
| +<enum name="AppInfoDialogLaunchSource" type="int"> |
| + <int value="0" label="App List context menu"/> |
| +</enum> |
| + |
| <enum name="AppLaunch" type="int"> |
| <int value="0" label="NTP_APPS_MAXIMIZED"/> |
| <int value="1" label="NTP_APPS_COLLAPSED"/> |
| @@ -35967,6 +35990,20 @@ Therefore, the affected-histogram name has to have at least one dot in it. |
| <int value="4" label="SEARCH_PEOPLE"/> |
| </enum> |
| +<enum name="AppLocation" type="int"> |
| + <int value="0" label="Invalid location"/> |
| + <int value="1" label="Internal extension"/> |
| + <int value="2" label="Internal extension (loaded via prefs)"/> |
| + <int value="3" label="Internal extension (loaded via the registry)"/> |
| + <int value="4" label="Unpacked extension"/> |
| + <int value="5" label="Component app"/> |
| + <int value="6" label="External extension (downloaded via prefs)"/> |
| + <int value="7" label="External extension (downloaded via admin policies)"/> |
| + <int value="8" label="Command-line extension"/> |
| + <int value="9" label="External extension (loaded via prefs and cached)"/> |
| + <int value="10" label="Component app (downloaded)"/> |
| +</enum> |
| + |
| <enum name="AppPromoAction" type="int"> |
| <int value="0" label="PROMO_LAUNCH_APP"/> |
| <int value="1" label="PROMO_LAUNCH_WEB_STORE"/> |
