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

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

Issue 2733833002: Change logic for recording redirect pattern histograms. (Closed)
Patch Set: fix deps Created 3 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
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 35bf3ee00b5c96f35104ccaf8a92476c6e6b2fea..a4cb8afc145c8575e7db12d87e15abcc913e3096 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -68716,12 +68716,16 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<histogram name="SubresourceFilter.PageLoad.RedirectChainLength" units="urls">
<owner>melandory@chromium.org</owner>
+ <suffix name="SocialEngineeringAdsInterstitial"/>
+ <suffix name="PhishingInterstitial"/>
<summary>Total length of the server redirects during the navigation.</summary>
</histogram>
<histogram name="SubresourceFilter.PageLoad.RedirectChainMatchPattern"
enum="SubresourceFilterMatchPattern">
<owner>melandory@chromium.org</owner>
+ <suffix name="SocialEngineeringAdsInterstitial"/>
+ <suffix name="PhishingInterstitial"/>
<summary>
For each main frame navigation, records a pattern that indicates which URLs
in the redirect chain matched Safe Browsing blacklists, and which did not.
@@ -122125,6 +122129,14 @@ from previous Chrome versions.
<affected-histogram name="WebRTC.Stun.SuccessPercent.UnknownNAT"/>
</histogram_suffixes>
+<histogram_suffixes name="SubresourceFilterOnlyExperiment" separator=".">
+ <suffix name="SocialEngineeringAdsInterstitial"/>
+ <suffix name="PhishingInterstitial"/>
+ <affected-histogram name="SubresourceFilter.PageLoad.RedirectChainLength"/>
+ <affected-histogram
+ name="SubresourceFilter.PageLoad.RedirectChainMatchPattern"/>
+</histogram_suffixes>
+
<histogram_suffixes name="SyncModelType" separator=".">
<suffix name="APP" label="APP"/>
<suffix name="APP_LIST" label="APP_LIST"/>

Powered by Google App Engine
This is Rietveld 408576698