Change logic for recording redirect pattern histograms.

tools/metrics/histograms/histograms.xml

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"/>

[jwd, 2017/03/15 14:10:38]

You don't need the suffix tags in the histogram tags, only in the
histogram_suffixes tag.

[melandory, 2017/03/15 14:21:33]

Sorry, I already fixed when this CL was second in a row =)

   <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"/>

[jwd, 2017/03/15 14:10:38]

Can you add a label to these suffixes, to make the expanded summary more
specific.

[melandory, 2017/03/15 14:21:33]

Done.

+  <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"/>