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

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

Issue 2897523004: Tab.NewTabOnload.Local histogram: Add split into Google/Other (Closed)
Patch Set: update histogram description Created 3 years, 7 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 | « chrome/browser/ui/search/search_tab_helper.cc ('k') | 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 22e2dbc85c346c590054c7191e3e9c858ed8153d..6b235df3c89033478557415a6bb7fc229dc4e3e8 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -74707,13 +74707,13 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</histogram>
<histogram name="Tab.NewTabOnload" units="ms">
- <owner>lliabraa@chromium.org</owner>
- <owner>beaudoin@chromium.org</owner>
+ <owner>treib@chromium.org</owner>
<summary>
The time for the new tab page to fire the &quot;load&quot; event. Note: This
- is usually recorded with a suffix (.Local/Google/Other). The base version is
- recorded only on Android, as well as for the old NTP (&quot;NTP4&quot;) on
- other platforms.
+ is usually recorded with a suffix (.Local*/Google/Other). The base version
+ is recorded only on Android, as well as for the old NTP (&quot;NTP4&quot;)
+ on other platforms. Recorded only once per tab, i.e. excluding back/forward
+ navigations.
</summary>
</histogram>
@@ -92725,6 +92725,9 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<histogram_suffixes name="TabNewTabOnload" separator=".">
<suffix name="Local" label="Local New Tab page."/>
+ <suffix name="LocalGoogle" label="Local New Tab page for Google."/>
+ <suffix name="LocalOther"
+ label="Local New Tab page for a non-Google provider."/>
<suffix name="Google" label="New Tab page for Google."/>
<suffix name="Other" label="New Tab page for a non-Google provider."/>
<affected-histogram name="Tab.NewTabOnload"/>
« no previous file with comments | « chrome/browser/ui/search/search_tab_helper.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698