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

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

Issue 2814733002: Add the SocEng as a type for checking in CheckUrlForSubresourceFilter. (Closed)
Patch Set: rebased and changes sync strategy 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
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 4771494ed098ff79098869cd46ec16b98d67350d..630ccbd641a3a877e3a3225b796f1d3a7d7e68db 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -71742,6 +71742,15 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="SubresourceFilter.PageLoad.FinalURLMatch" units="matches">
+ <owner>melandory@chromium.org</owner>
+ <summary>
+ Records, for each main frame navigation, whether the last URL in the
+ redirect chain matched the Safe Browsing blacklist specified by the
+ histogram suffix.
+ </summary>
+</histogram>
+
<histogram name="SubresourceFilter.PageLoad.NumSubresourceLoads.Disallowed"
units="resource loads">
<owner>pkalinnikov@chromium.org</owner>
@@ -71795,6 +71804,10 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<histogram name="SubresourceFilter.PageLoad.RedirectChainMatchPattern"
enum="SubresourceFilterMatchPattern">
+ <obsolete>
+ Obsolete as of April 2017, since the don't have correct data to record
+ anymore.
+ </obsolete>
<owner>melandory@chromium.org</owner>
<summary>
For each main frame navigation, records a pattern that indicates which URLs
@@ -91859,10 +91872,15 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<suffix name="SocialEngineeringAdsInterstitial"
label="social eng ad blacklist pattern"/>
<suffix name="PhishingInterstitial" label="phishing blacklist pattern"/>
- <suffix name="SubresourceFilterOnly" label="subresource filter only patern"/>
+ <suffix name="SubresourceFilterOnly" label="subresource filter only pattern"/>
+ <affected-histogram name="SubresourceFilter.PageLoad.FinalURLMatch"/>
<affected-histogram name="SubresourceFilter.PageLoad.RedirectChainLength"/>
<affected-histogram
- name="SubresourceFilter.PageLoad.RedirectChainMatchPattern"/>
+ name="SubresourceFilter.PageLoad.RedirectChainMatchPattern">
+ <obsolete>
+ Obsolete as of April 2017
+ </obsolete>
+ </affected-histogram>
</histogram_suffixes>
<histogram_suffixes name="SyncModelType" separator=".">

Powered by Google App Engine
This is Rietveld 408576698