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

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

Side-by-side diff isn't available for this file because of its large size.
Issue 475283002: Add owners for apps and app list related histograms (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 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:
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 071496f021bfb82e9438ec315a9f5f66d4bd6b18..7f128f431ed8207317ee91932caf77936aec14b0 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -419,7 +419,8 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</histogram>
<histogram name="Apps.AppLaunch" enum="AppLaunch">
- <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
+ <owner>tapted@chromium.org</owner>
+ <owner>benwells@chromium.org</owner>
<summary>
The number of times v2 packaged apps are launched grouped by
extension_misc::AppLaunchBuckets. See also Extensions.AppLaunch.
@@ -432,7 +433,7 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</histogram>
<histogram name="Apps.AppListHowEnabled" enum="AppListEnableSource">
- <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
+ <owner>tapted@chromium.org</owner>
<summary>
The trigger that caused the app list to be enabled. Recorded when the user
first shows the app list. If not shown after one hour, will be recorded
@@ -442,7 +443,7 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</histogram>
<histogram name="Apps.AppListSearchCommenced" units="searches">
- <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
+ <owner>tapted@chromium.org</owner>
<summary>
The number of searches that are started in the app list. This is gathered
each time the app list search box transitions from empty to non-empty.
@@ -450,7 +451,7 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</histogram>
<histogram name="Apps.AppListSearchResultOpenType" enum="AppListSearchResult">
- <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
+ <owner>tapted@chromium.org</owner>
<summary>
The type of app list search result that was opened by the user. This is
gathered per click of a search result.
@@ -458,7 +459,7 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</histogram>
<histogram name="Apps.AppListTimeToDiscover" units="milliseconds">
- <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
+ <owner>tapted@chromium.org</owner>
<summary>
Time between enabling the app list, and a user explicitly choosing to show
it. If the app list is not shown after one hour, an entry in the last bucket
@@ -7043,7 +7044,8 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</histogram>
<histogram name="Extensions.AppLaunch" enum="AppLaunch">
- <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
+ <owner>benwells@chromium.org</owner>
+ <owner>tapted@chromium.org</owner>
<summary>
The number of times v1 apps are launched grouped by
extension_misc::AppLaunchBuckets. See also Apps.AppLaunch for v2 apps.
@@ -7051,7 +7053,8 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</histogram>
<histogram name="Extensions.AppLaunchContainer" enum="AppLaunchContainer">
- <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
+ <owner>benwells@chromium.org</owner>
+ <owner>tapted@chromium.org</owner>
<summary>
The number of times apps are launched grouped by
extensions::LaunchContainer.
@@ -7059,14 +7062,16 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</histogram>
<histogram name="Extensions.AppLocation" enum="ExtensionLocation">
- <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
+ <owner>benwells@chromium.org</owner>
+ <owner>tapted@chromium.org</owner>
<summary>
The number of apps loaded at startup time grouped by Extension::Location.
</summary>
</histogram>
<histogram name="Extensions.AppsPromo" enum="AppPromoAction">
- <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
+ <owner>benwells@chromium.org</owner>
+ <owner>tapted@chromium.org</owner>
<summary>
The actions taken in the NTP apps promo grouped by
extension_misc::AppsPromoBuckets.
@@ -7074,7 +7079,8 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</histogram>
<histogram name="Extensions.AppTabLaunchType" enum="ExtensionLaunchType">
- <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
+ <owner>benwells@chromium.org</owner>
+ <owner>tapted@chromium.org</owner>
<summary>
The number of apps launched grouped by extensions::LaunchType.
</summary>
@@ -31605,6 +31611,7 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<histogram name="Startup.ShowAppListColdStart" units="milliseconds">
<owner>jeremy@chromium.org</owner>
+ <owner>tapted@chromium.org</owner>
<summary>
Time for a newly created browser process to reach the code that starts
showing the app launcher, when started with the --show-app-list flag and
@@ -31614,6 +31621,7 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<histogram name="Startup.ShowAppListWarmStart" units="milliseconds">
<owner>jeremy@chromium.org</owner>
+ <owner>tapted@chromium.org</owner>
<summary>
Time for a running browser process to reach the code that starts showing the
app launcher. Measured from the time a second Chrome process started, which
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698