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

Unified Diff: chrome/browser/renderer_host/safe_browsing_resource_throttle.cc

Issue 1515703005: WebRequest API: add more resource types (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Replace == with && 2x Created 5 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:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/renderer_host/safe_browsing_resource_throttle.cc
diff --git a/chrome/browser/renderer_host/safe_browsing_resource_throttle.cc b/chrome/browser/renderer_host/safe_browsing_resource_throttle.cc
index 5840ed7a798bb77a9681b1f6cab8481c95f43dd0..368908d4c899561ff738bb5b73237758672af2b5 100644
--- a/chrome/browser/renderer_host/safe_browsing_resource_throttle.cc
+++ b/chrome/browser/renderer_host/safe_browsing_resource_throttle.cc
@@ -31,7 +31,7 @@ namespace {
const int kCheckUrlTimeoutMs = 5000;
void RecordHistogramResourceTypeSafe(content::ResourceType resource_type) {
- UMA_HISTOGRAM_ENUMERATION("SB2.ResourceTypes.Safe", resource_type,
+ UMA_HISTOGRAM_ENUMERATION("SB2.ResourceTypes2.Safe", resource_type,
content::RESOURCE_TYPE_LAST_TYPE);
}
@@ -233,12 +233,12 @@ void SafeBrowsingResourceThrottle::OnCheckBrowseUrlResult(
if (request_->load_flags() & net::LOAD_PREFETCH) {
// Don't prefetch resources that fail safe browsing, disallow them.
controller()->Cancel();
- UMA_HISTOGRAM_ENUMERATION("SB2.ResourceTypes.UnsafePrefetchCanceled",
+ UMA_HISTOGRAM_ENUMERATION("SB2.ResourceTypes2.UnsafePrefetchCanceled",
resource_type_, content::RESOURCE_TYPE_LAST_TYPE);
return;
}
- UMA_HISTOGRAM_ENUMERATION("SB2.ResourceTypes.Unsafe", resource_type_,
+ UMA_HISTOGRAM_ENUMERATION("SB2.ResourceTypes2.Unsafe", resource_type_,
content::RESOURCE_TYPE_LAST_TYPE);
const content::ResourceRequestInfo* info =
@@ -323,13 +323,13 @@ bool SafeBrowsingResourceThrottle::CheckUrl(const GURL& url) {
// To reduce aggregate latency on mobile, check only the most dangerous
// resource types.
if (!database_manager_->CanCheckResourceType(resource_type_)) {
- UMA_HISTOGRAM_ENUMERATION("SB2.ResourceTypes.Skipped", resource_type_,
+ UMA_HISTOGRAM_ENUMERATION("SB2.ResourceTypes2.Skipped", resource_type_,
content::RESOURCE_TYPE_LAST_TYPE);
return true;
}
bool succeeded_synchronously = database_manager_->CheckBrowseUrl(url, this);
- UMA_HISTOGRAM_ENUMERATION("SB2.ResourceTypes.Checked", resource_type_,
+ UMA_HISTOGRAM_ENUMERATION("SB2.ResourceTypes2.Checked", resource_type_,
content::RESOURCE_TYPE_LAST_TYPE);
if (succeeded_synchronously) {

Powered by Google App Engine
This is Rietveld 408576698