Chromium Code Reviews| Index: tools/metrics/histograms/histograms.xml |
| diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml |
| index fb1a1d0bd2d615446f53471e964310647397cb38..20b145be4b6ad69fc0e0ea42c88dfffcd7df999e 100644 |
| --- a/tools/metrics/histograms/histograms.xml |
| +++ b/tools/metrics/histograms/histograms.xml |
| @@ -9990,7 +9990,7 @@ Therefore, the affected-histogram name has to have at least one dot in it. |
| <histogram name="InstantExtended.CacheableNTPLoad" |
| enum="InstantExtended_CacheableNTPLoad"> |
| - <owner>Please list the metric's owners. Add more owner tags as needed.</owner> |
| + <owner>beaudoin@chromium.org</owner> |
| <summary> |
| Records a histogram for how often the Cacheable NTP fails to load. |
| </summary> |
| @@ -19572,7 +19572,7 @@ Therefore, the affected-histogram name has to have at least one dot in it. |
| </histogram> |
| <histogram name="NewTabPage.URLState" enum="NewTabURLState"> |
| - <owner>Please list the metric's owners. Add more owner tags as needed.</owner> |
| + <owner>beaudoin@chromium.org</owner> |
| <summary> |
| Records the status of the New Tab page URL when an NTP is opened. |
| </summary> |
| @@ -33407,6 +33407,42 @@ Therefore, the affected-histogram name has to have at least one dot in it. |
| </summary> |
| </histogram> |
| +<histogram name="Tab.NewTab" enum="NewTabType"> |
|
Alexei Svitkine (slow)
2014/08/15 14:59:42
Please also send me a CL removing these histograms
|
| + <owner>lliabraa@chromium.org</owner> |
| + <owner>beaudoin@chromium.org</owner> |
| + <summary> |
| + Tracks the different ways users are opening new tabs. Does not apply to |
| + opening existing links or searches in a new tab, only to brand new empty |
| + tabs. Note: Currently the "Regular menu option" includes some |
| + programmatic actions in addition to user actions. |
| + </summary> |
| +</histogram> |
| + |
| +<histogram name="Tab.NewTabDOMContentLoaded" units="milliseconds"> |
| + <owner>lliabraa@chromium.org</owner> |
| + <owner>beaudoin@chromium.org</owner> |
| + <summary> |
| + The time for the new tab page to fire the "DOMContentLoaded" |
| + event. |
| + </summary> |
| +</histogram> |
| + |
| +<histogram name="Tab.NewTabOnload" units="milliseconds"> |
| + <owner>lliabraa@chromium.org</owner> |
| + <owner>beaudoin@chromium.org</owner> |
| + <summary> |
| + The time for the new tab page to fire the "load" event. |
| + </summary> |
| +</histogram> |
| + |
| +<histogram name="Tab.NewTabScriptStart" units="milliseconds"> |
| + <owner>lliabraa@chromium.org</owner> |
| + <owner>beaudoin@chromium.org</owner> |
| + <summary> |
| + The time for the new tab page to start executing JavaScript. |
| + </summary> |
| +</histogram> |
| + |
| <histogram name="Tab.PerceivedRestoreTime" units="ms"> |
| <owner>lliabraa@chromium.org</owner> |
| <summary> |
| @@ -45141,6 +45177,12 @@ Therefore, the affected-histogram name has to have at least one dot in it. |
| <int value="4" label="User dismissed the promo"/> |
| </enum> |
| +<enum name="NewTabType" type="int"> |
| + <int value="0" label="New tab button"/> |
| + <int value="1" label="Regular menu option"/> |
| + <int value="2" label="Tab strip menu option"/> |
| +</enum> |
| + |
| <enum name="NewTabURLState" type="int"> |
| <int value="0" label="Valid URL was used"/> |
| <int value="1" label="Corrupt state"/> |
| @@ -53824,6 +53866,13 @@ Therefore, the affected-histogram name has to have at least one dot in it. |
| <affected-histogram name="Startup.BrowserOpenTabs"/> |
| </histogram_suffixes> |
| +<histogram_suffixes name="TabNewTabOnload" separator="."> |
| + <suffix name="Local" label="Local New Tab page."/> |
| + <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"/> |
| +</histogram_suffixes> |
| + |
| <histogram_suffixes name="Tps65090Fets" separator="."> |
| <suffix name="Fet1" label="FET1 on tps65090 (register 0xf)"/> |
| <suffix name="Fet2" label="FET2 on tps65090 (register 0x10)"/> |