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

Unified Diff: chrome/browser/safe_browsing/incident_reporting/blacklist_load_incident_handlers.cc

Issue 433513008: Add blacklist incidents to incident reporting service. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: mattm feedback Created 6 years, 4 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/incident_reporting/blacklist_load_incident_handlers.cc
diff --git a/chrome/browser/safe_browsing/incident_reporting/binary_integrity_incident_handlers.cc b/chrome/browser/safe_browsing/incident_reporting/blacklist_load_incident_handlers.cc
similarity index 52%
copy from chrome/browser/safe_browsing/incident_reporting/binary_integrity_incident_handlers.cc
copy to chrome/browser/safe_browsing/incident_reporting/blacklist_load_incident_handlers.cc
index 0ad0c8505f1bb09765d0ce2915524f377ce787e9..1847837211cdca3cff556493d71b362e1f374c73 100644
--- a/chrome/browser/safe_browsing/incident_reporting/binary_integrity_incident_handlers.cc
+++ b/chrome/browser/safe_browsing/incident_reporting/blacklist_load_incident_handlers.cc
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "chrome/browser/safe_browsing/incident_reporting/binary_integrity_incident_handlers.h"
+#include "chrome/browser/safe_browsing/incident_reporting/blacklist_load_incident_handlers.h"
#include "base/logging.h"
#include "chrome/browser/safe_browsing/incident_reporting/incident_handler_util.h"
@@ -10,17 +10,17 @@
namespace safe_browsing {
-std::string GetBinaryIntegrityIncidentKey(
+std::string GetBlacklistLoadIncidentKey(
const ClientIncidentReport_IncidentData& incident_data) {
- DCHECK(incident_data.has_binary_integrity());
- DCHECK(incident_data.binary_integrity().has_file_basename());
- return incident_data.binary_integrity().file_basename();
+ DCHECK(incident_data.has_blacklist_load());
+ DCHECK(incident_data.blacklist_load().has_path());
+ return incident_data.blacklist_load().path();
}
-uint32_t GetBinaryIntegrityIncidentDigest(
+uint32_t GetBlacklistLoadIncidentDigest(
const ClientIncidentReport_IncidentData& incident_data) {
- DCHECK(incident_data.has_binary_integrity());
- return HashMessage(incident_data.binary_integrity());
+ DCHECK(incident_data.has_blacklist_load());
+ return HashMessage(incident_data.blacklist_load());
}
} // namespace safe_browsing

Powered by Google App Engine
This is Rietveld 408576698