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

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

Side-by-side diff isn't available for this file because of its large size.
Issue 263833005: Knock out injected safe-browsing prefixes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: additional color on the histogram. Created 6 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
« no previous file with comments | « chrome/browser/safe_browsing/safe_browsing_store_unittest.cc ('k') | no next file » | 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 6a5af869050fbc916b9272440c9483b9f588ff7b..2e3e29210c07b658e3643ab57bc4c11aa352ae0a 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -22418,6 +22418,18 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<summary>The number of chunk URLs in an update response.</summary>
</histogram>
+<histogram name="SB2.VolunteerPrefixesRemoved">
+ <owner>shess@chromium.org</owner>
+ <summary>
+ Older versions of the safe-browsing code incorrectly added additional
+ SBPrefix items when receiving full hashes. This caused errors when
+ calculating when to send gethash requests to the server. An additional pass
+ over the data has been added to remove the excess prefixes. This histogram
+ tracks progress of that code for purposes of informing a decision on when to
+ remove the additional pass. See http://crbug.com/361248 .
+ </summary>
+</histogram>
+
<histogram name="SBClientDownload.CheckDownloadStats"
enum="SBClientDownloadCheckDownloadStats">
<owner>mattm@chromium.org</owner>
« no previous file with comments | « chrome/browser/safe_browsing/safe_browsing_store_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698