commit 7d9f770a5a163a4981cf936b5e18c97b2f86db64
author Charles Harrison <csharrison@chromium.org> Fri Oct 20 19:45:05 2017
committer Commit Bot <commit-bot@chromium.org> Fri Oct 20 19:45:05 2017
tree 28a54288255e1df39fb900a3ec915503300f4ab1
parent 424c10d86c835e5b1eb7b90716848f7bb34987cb

[subresource_filter] Remove SubresourceFilter.PageLoad.BlockedPopups

This metric is broken, and we are moving to a new system for blocking
popups anyway.

The new metric will be landed here:
https://chromium-review.googlesource.com/c/chromium/src/+/726545

Bug: 772946
Change-Id: Ib6ce9d316f7188be13a53988e06623ee93065df3
Reviewed-on: https://chromium-review.googlesource.com/728953
Reviewed-by: Shivani Sharma <shivanisha@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Commit-Queue: Charlie Harrison <csharrison@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#510525}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 655b3cccf128cb8ee6f000a871ff0726b6134ca9

metrics/histograms/histograms.xml

diff --git a/metrics/histograms/histograms.xml b/metrics/histograms/histograms.xml
index 2b28963..8741cec 100644
--- a/metrics/histograms/histograms.xml
+++ b/metrics/histograms/histograms.xml
@@ -81926,6 +81926,12 @@
 </histogram>
 
 <histogram name="SubresourceFilter.PageLoad.BlockedPopups" units="popups">
+  <obsolete>
+    Metric is broken and will be replaced by
+    ContentSettings.Popups.StringBlocked.NumBlocked. This only logged values for
+    sites which are also activated for subresource filtering, when most sites
+    will probably only be marked for popup blocking.
+  </obsolete>
   <owner>csharrison@chromium.org</owner>
   <summary>
     Records the number of popups blocked per page load due to subresource filter