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

Unified Diff: components/safe_browsing_db/v4_local_database_manager.cc

Issue 2648193002: SafeBrowsing: Move enabled_ to base class DB Manager (Closed)
Patch Set: Fix compile error Created 3 years, 11 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: components/safe_browsing_db/v4_local_database_manager.cc
diff --git a/components/safe_browsing_db/v4_local_database_manager.cc b/components/safe_browsing_db/v4_local_database_manager.cc
index ffef84ef0258cb080a409c3a88921a642e87d6c6..42e8f6319c4da539f359dec68b78f6dad621de24 100644
--- a/components/safe_browsing_db/v4_local_database_manager.cc
+++ b/components/safe_browsing_db/v4_local_database_manager.cc
@@ -138,7 +138,6 @@ scoped_refptr<V4LocalDatabaseManager> V4LocalDatabaseManager::Create(
V4LocalDatabaseManager::V4LocalDatabaseManager(const base::FilePath& base_path)
: base_path_(base_path),
- enabled_(false),
list_infos_(GetListInfos()),
weak_factory_(this) {
DCHECK(!base_path_.empty());
@@ -368,15 +367,11 @@ void V4LocalDatabaseManager::StartOnIOThread(
SetupUpdateProtocolManager(request_context_getter, config);
SetupDatabase();
-
- enabled_ = true;
}
void V4LocalDatabaseManager::StopOnIOThread(bool shutdown) {
DCHECK_CURRENTLY_ON(BrowserThread::IO);
- enabled_ = false;
-
pending_checks_.clear();
RespondSafeToQueuedChecks();

Powered by Google App Engine
This is Rietveld 408576698