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

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

Side-by-side diff isn't available for this file because of its large size.
Issue 1007383003: Adds a new UMA histogram to measure the NTP load time since navigation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 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/common/ntp_logging_events.h ('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 35c3c071a86a9d8d52a18c6df008a67fe2997faa..ebb1ccc431173aa59110a67f34917a55a40ac745 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -8895,9 +8895,8 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<histogram name="Extensions.FeatureProviderStaticInitTime" units="milliseconds">
<owner>rkaplow@chromium.org</owner>
<summary>
- The amount of time that elapsed during
- extensions::FeatureProvider::Static. Only measured while in the browser
- process.
+ The amount of time that elapsed during extensions::FeatureProvider::Static.
+ Only measured while in the browser process.
</summary>
</histogram>
@@ -19607,8 +19606,8 @@ Therefore, the affected-histogram name has to have at least one dot in it.
enum="QuicSessionLocations">
<owner>rch@chromium.org</owner>
<summary>
- The location in quic_client_session.cc where a session is unexpectedly
- not going away.
+ The location in quic_client_session.cc where a session is unexpectedly not
+ going away.
</summary>
</histogram>
@@ -23072,6 +23071,14 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</summary>
</histogram>
+<histogram name="NewTabPage.LoadTime">
jwd 2015/03/17 17:08:11 Can you add units="milliseconds" please.
fserb 2015/03/17 17:10:11 Done.
+ <owner>fserb@chromium.org</owner>
+ <summary>
+ Histogram of the time, in milliseconds, it took for the NTP to load all its
jwd 2015/03/17 17:08:11 nit: all of its
fserb 2015/03/17 17:10:11 Done.
+ tiles since navigation start.
+ </summary>
+</histogram>
+
<histogram name="NewTabPage.MobilePromo" enum="NewTabPageMobilePromo">
<obsolete>
Deprecated on M33 with the change to native NTP.
@@ -23138,9 +23145,9 @@ Therefore, the affected-histogram name has to have at least one dot in it.
The number of tiles for which we relied on external tiles as a fallback
because a local screenshot was not available to be used as a thumbnail.
External tiles are those for which the visuals are handled by the page
- itself, not by the iframe. Recorded before changing focus away from the NTP,
- be it bynavigating to a URL, switching tabs, changing the active window or
- closing the tab/shutting down Chrome.
+ itself, not by the iframe. Recorded before reloading the suggestions,
+ navigating to a URL, switching tabs, changing the active window, or closing
+ the tab/shutting down Chrome.
</summary>
</histogram>
@@ -23149,9 +23156,9 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<summary>
The number of external tiles that are displayed on the NTP. External tiles
are those for which the visuals are handled by the page itself, not by the
- iframe. Recorded before changing focus away from the NTP, be it by
- navigating to a URL, switching tabs, changing the active window or closing
- the tab/shutting down Chrome.
+ iframe. Recorded before reloading the suggestions, navigating to a URL,
+ switching tabs, changing the active window or closing the tab/shutting down
+ Chrome.
</summary>
</histogram>
@@ -23160,9 +23167,9 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<summary>
The number of tiles for which we displayed a gray tile with the domain name
as a fallback because a local screenshot was not available to be used as a
- thumbnail. Recorded before changing focus away from the NTP, be it by
- navigating to a URL, switching tabs, changing the active window or closing
- the tab/shutting down Chrome.
+ thumbnail. Recorded before reloading the suggestions, navigating to a URL,
+ switching tabs, changing the active window or closing the tab/shutting down
+ Chrome.
</summary>
</histogram>
@@ -23171,8 +23178,8 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<summary>
The number of tiles for which no thumbnail was specified, but a domain was
so we displayed a gray tile with the domain name in it. Recorded before
- changing focus away from the NTP, be it by navigating to a URL, switching
- tabs, changing the active window or closing the tab/shutting down Chrome.
+ reloading the suggestions, navigating to a URL, switching tabs, changing the
+ active window or closing the tab/shutting down Chrome.
</summary>
</histogram>
@@ -23193,9 +23200,9 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<owner>Please list the metric's owners. Add more owner tags as needed.</owner>
<summary>
The number of tiles for which we attempted to use a local screenshot as a
- thumbnail. Recorded before changing focus away from the NTP, be it by
- navigating to a URL, switching tabs, changing the active window or closing
- the tab/shutting down Chrome.
+ thumbnail. Recorded before reloading the suggestions, navigating to a URL,
+ switching tabs, changing the active window or closing the tab/shutting down
+ Chrome.
</summary>
</histogram>
@@ -23204,9 +23211,8 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<summary>
The number of thumbnails for which a local screenshot was not available so
we were not able to display them on the Most Visited section of the NTP.
- Recorded before changing focus away from the NTP, be it by navigating to a
- URL, switching tabs, changing the active window or closing the tab/shutting
- down Chrome.
+ Recorded before reloading the suggestions, navigating to a URL, switching
+ tabs, changing the active window or closing the tab/shutting down Chrome.
</summary>
</histogram>
@@ -23214,9 +23220,9 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<owner>beaudoin@chromium.org</owner>
<summary>
The number of tiles for which we attempted to use a local screenshot as a
- thumbnail. Recorded before changing focus away from the NTP, be it by
- navigating to a URL, switching tabs, changing the active window or closing
- the tab/shutting down Chrome.
+ thumbnail. Recorded before reloading the suggestions, navigating to a URL,
+ switching tabs, changing the active window or closing the tab/shutting down
+ Chrome.
</summary>
</histogram>
@@ -23224,9 +23230,9 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<owner>beaudoin@chromium.org</owner>
<summary>
The number of tiles that are displayed on the NTP, no matter if they are
- thumbnails, gray tiles, or external tiles. Recorded before changing focus
- away from the NTP, be it by navigating to a URL, switching tabs, changing
- the active window or closing the tab/shutting down Chrome.
+ thumbnails, gray tiles, or external tiles. Recorded before reloading the
+ suggestions, navigating to a URL, switching tabs, changing the active window
+ or closing the tab/shutting down Chrome.
</summary>
</histogram>
« no previous file with comments | « chrome/common/ntp_logging_events.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698