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

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

Issue 2787573003: Remove DumpWithoutCrashing from ShouldAllowOpenURL. (Closed)
Patch Set: Address asvitkine's comments Created 3 years, 8 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 | « content/public/common/content_client.h ('k') | tools/metrics/histograms/presubmit_scheme_histograms.py » ('j') | 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 becf04e8626b9ab2d5a2c0e5fe563a5a9e575a93..38abc29f32e747d8fc4e2b470a3f5a634e557931 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -19848,6 +19848,18 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="Extensions.ShouldAllowOpenURL.Failure.Scheme"
+ enum="ShouldAllowOpenURLFailureScheme">
+ <owner>alexmos@chromium.org</owner>
+ <summary>
+ When the web-accessible resource check in
+ ChromeContentBrowserClientExtensionsPart::ShouldAllowOpenURL fails, this
+ records the scheme of the SiteInstance that initiated the blocked load. This
+ check is performed on navigations that utilize the OpenURL path as well as
+ on transfers.
+ </summary>
+</histogram>
+
<histogram name="Extensions.StartupDelay" units="ms">
<owner>asargent@chromium.org</owner>
<summary>The time one extension delays network requests at startup.</summary>
@@ -110179,6 +110191,31 @@ from previous Chrome versions.
<int value="3" label="Resource is not web-accessible (most common)"/>
</enum>
+<enum name="ShouldAllowOpenURLFailureScheme" type="int">
+<!-- Generated from chrome/browser/extensions/chrome_content_browser_client_extensions_part.cc -->
+
+ <int value="0" label="SCHEME_UNKNOWN"/>
+ <int value="1" label="SCHEME_EMPTY"/>
+ <int value="2" label="SCHEME_HTTP"/>
+ <int value="3" label="SCHEME_HTTPS"/>
+ <int value="4" label="SCHEME_FILE"/>
+ <int value="5" label="SCHEME_FTP"/>
+ <int value="6" label="SCHEME_DATA"/>
+ <int value="7" label="SCHEME_JAVASCRIPT"/>
+ <int value="8" label="SCHEME_ABOUT"/>
+ <int value="9" label="SCHEME_CHROME"/>
+ <int value="10" label="SCHEME_DEVTOOLS"/>
+ <int value="11" label="SCHEME_GUEST"/>
+ <int value="12" label="SCHEME_VIEWSOURCE"/>
+ <int value="13" label="SCHEME_CHROME_SEARCH"/>
+ <int value="14" label="SCHEME_CHROME_NATIVE"/>
+ <int value="15" label="SCHEME_DOM_DISTILLER"/>
+ <int value="16" label="SCHEME_CHROME_EXTENSION"/>
+ <int value="17" label="SCHEME_CONTENT"/>
+ <int value="18" label="SCHEME_BLOB"/>
+ <int value="19" label="SCHEME_FILESYSTEM"/>
+</enum>
+
<enum name="ShutdownReason" type="int">
<summary>
The reason that the Chrome OS power manager shut down or rebooted the
« no previous file with comments | « content/public/common/content_client.h ('k') | tools/metrics/histograms/presubmit_scheme_histograms.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698