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

Unified Diff: chrome/browser/safe_browsing/client_side_detection_service.cc

Issue 173133004: Separate pre-classification checks for client-side malware and phishing (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Set right enum type for boolean histograms. Created 6 years, 9 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:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/safe_browsing/client_side_detection_service.cc
diff --git a/chrome/browser/safe_browsing/client_side_detection_service.cc b/chrome/browser/safe_browsing/client_side_detection_service.cc
index d63e45cbfd931b57fd880f4d48c40c168454bbbe..d045bd92f986bdb383d437c264147a2dcbc2c2b4 100644
--- a/chrome/browser/safe_browsing/client_side_detection_service.cc
+++ b/chrome/browser/safe_browsing/client_side_detection_service.cc
@@ -368,15 +368,6 @@ void ClientSideDetectionService::StartClientReportMalwareRequest(
return;
}
- if (OverMalwareReportLimit()) {
- UpdateEnumUMAHistogram(REPORT_HIT_LIMIT);
- DVLOG(1) << "Too many malware report requests sent recently."
- << "Skip sending malware report for " << GURL(request->url());
- if (!callback.is_null())
- callback.Run(GURL(request->url()), GURL(request->url()), false);
- return;
- }
-
std::string request_data;
if (!request->SerializeToString(&request_data)) {
UpdateEnumUMAHistogram(REPORT_FAILED_SERIALIZATION);

Powered by Google App Engine
This is Rietveld 408576698