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

Unified Diff: chrome/browser/safe_browsing/local_database_manager.h

Issue 2505153002: Add support for scout to 'ext' param when creating SafeBrowsing ping URLs. (Closed)
Patch Set: Go back to IsExtendedReportingEnabled where level doesn't matter Created 4 years, 1 month 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/local_database_manager.h
diff --git a/chrome/browser/safe_browsing/local_database_manager.h b/chrome/browser/safe_browsing/local_database_manager.h
index e78abcf094f75944395834561fe5be5663d3600b..521770e091ddbaa2b9693980c8e1c585d409abe8 100644
--- a/chrome/browser/safe_browsing/local_database_manager.h
+++ b/chrome/browser/safe_browsing/local_database_manager.h
@@ -28,6 +28,7 @@
#include "chrome/browser/safe_browsing/protocol_manager.h"
#include "chrome/browser/safe_browsing/safe_browsing_util.h"
#include "components/safe_browsing_db/database_manager.h"
+#include "components/safe_browsing_db/safe_browsing_prefs.h"
#include "components/safe_browsing_db/safebrowsing.pb.h"
#include "components/safe_browsing_db/util.h"
#include "url/gurl.h"
@@ -80,7 +81,7 @@ class LocalSafeBrowsingDatabaseManager
std::vector<SBThreatType> full_hash_results;
SafeBrowsingDatabaseManager::Client* client;
- bool is_extended_reporting;
+ ExtendedReportingLevel extended_reporting_level;
bool need_get_hash;
base::TimeTicks start; // When check was sent to SB service.
ListType check_type; // See comment in constructor.
@@ -216,9 +217,9 @@ class LocalSafeBrowsingDatabaseManager
// Called on the UI thread to prepare hash request.
void OnRequestFullHash(SafeBrowsingCheck* check);
- // Called on the UI thread to determine if current profile is opted into
- // extended reporting.
- bool GetExtendedReporting();
+ // Called on the UI thread to determine what level of extended reporting the
+ // current profile is opted into.
+ ExtendedReportingLevel GetExtendedReporting();
// Called on the IO thread to request full hash.
void RequestFullHash(SafeBrowsingCheck* check);
@@ -235,7 +236,7 @@ class LocalSafeBrowsingDatabaseManager
// Called on the IO thread with the results of all chunks.
void OnGetAllChunksFromDatabase(const std::vector<SBListChunkRanges>& lists,
bool database_error,
- bool is_extended_reporting,
+ ExtendedReportingLevel reporting_level,
GetChunksCallback callback);
// Called on the IO thread after the database reports that it added a chunk.
« no previous file with comments | « chrome/browser/safe_browsing/download_protection_service.cc ('k') | chrome/browser/safe_browsing/local_database_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698