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

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

Side-by-side diff isn't available for this file because of its large size.
Issue 790703003: De-dup PrefixSet code in SafeBrowsingDatabaseManager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@a5_rm_unused_extensionsBL
Patch Set: rebase onto 744183002 Created 6 years 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 024435dd002a22cd42b56d8971bde44cf4e2c1c9..c0336cb2ee2ad91692d0da8fa252b702813aae90 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -29765,12 +29765,10 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</histogram>
<histogram name="SB2.DatabaseKilobytes" units="KB">
- <obsolete>
- Replaced by SB2.BrowseDatabaseKilobytes.
- </obsolete>
- <owner>Please list the metric's owners. Add more owner tags as needed.</owner>
+ <owner>gab@chromium.org</owner>
<summary>
- The size of the SafeBrowsing database file on disk in kilobytes.
+ The size of one of the browsing SafeBrowsing database file on disk in
+ kilobytes, after an update has occurred.
</summary>
</histogram>
@@ -30216,12 +30214,12 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<histogram name="SB2.PrefixSetKilobytes" units="KB">
<owner>shess@chromium.org</owner>
- <summary>The size of the PrefixSet file in kilobytes.</summary>
+ <summary>The size of one of the PrefixSet files in kilobytes.</summary>
</histogram>
<histogram name="SB2.PrefixSetLoad" units="ms">
<owner>shess@chromium.org</owner>
- <summary>Time to load the PrefixSet file.</summary>
+ <summary>Time to load one of the PrefixSet files.</summary>
</histogram>
<histogram name="SB2.PrefixSetRestoredExcess">
@@ -30289,12 +30287,12 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<histogram name="SB2.PrefixSetVersionRead">
<owner>shess@chromium.org</owner>
- <summary>Version read from the PrefixSet file.</summary>
+ <summary>Version read from one of the PrefixSet files.</summary>
</histogram>
<histogram name="SB2.PrefixSetWrite" units="ms">
<owner>shess@chromium.org</owner>
- <summary>Time to store the PrefixSet file.</summary>
+ <summary>Time to store one of the PrefixSet files.</summary>
</histogram>
<histogram name="SB2.ReportingIsEnabled" enum="BooleanEnabled">
@@ -30328,7 +30326,23 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</summary>
</histogram>
+<histogram name="SB2.SideEffectFreePrefixSetWrite" units="ms">
+ <obsolete>
+ Deprecated 12/2014. Merged into SB2.PrefixSetWrite.
+ </obsolete>
+ <owner>shess@chromium.org</owner>
+ <summary>
+ Time to store the Side Effect Free Whitelist PrefixSet file. Note: this
+ histogram was intended to be stored as
+ SB2.SideEffectFreeWhitelistPrefixSetWrite but was actually reported as
+ SB2.SideEffectFreePrefixSetWrite from its inception to its deprecation...
+ </summary>
+</histogram>
+
<histogram name="SB2.SideEffectFreeWhitelistDatabaseKilobytes" units="KB">
+ <obsolete>
+ Deprecated 12/2014. Merged into SB2.DatabaseKilobytes.
+ </obsolete>
<owner>shess@chromium.org</owner>
<summary>
The size of the Side Effect Free Whitelist SaafeBrowsing database file on
@@ -30337,6 +30351,9 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</histogram>
<histogram name="SB2.SideEffectFreeWhitelistPrefixSetKilobytes" units="KB">
+ <obsolete>
+ Deprecated 12/2014. Merged into SB2.PrefixSetKilobytes.
+ </obsolete>
<owner>shess@chromium.org</owner>
<summary>
The size of the Side Effect Free Whitelist PrefixSet file in kilobytes,
@@ -30345,17 +30362,13 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</histogram>
<histogram name="SB2.SideEffectFreeWhitelistPrefixSetLoad" units="ms">
+ <obsolete>
+ Deprecated 12/2014. Merged into SB2.PrefixSetLoad.
+ </obsolete>
<owner>shess@chromium.org</owner>
<summary>Time to load the Side Effect Free Whitelist PrefixSet file.</summary>
</histogram>
-<histogram name="SB2.SideEffectFreeWhitelistPrefixSetWrite" units="ms">
- <owner>shess@chromium.org</owner>
- <summary>
- Time to store the Side Effect Free Whitelist PrefixSet file.
- </summary>
-</histogram>
-
<histogram name="SB2.SideEffectFreeWhitelistStatus"
enum="SB2SideEffectFreeWhitelistStatus">
<owner>shess@chromium.org</owner>

Powered by Google App Engine
This is Rietveld 408576698