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

Unified Diff: components/safe_browsing_db/v4_local_database_manager.h

Issue 2345573002: Each DatabaseManager gets to decide which stores to track (Closed)
Patch Set: Incorporated nparker@ feedback Created 4 years, 3 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.h
diff --git a/components/safe_browsing_db/v4_local_database_manager.h b/components/safe_browsing_db/v4_local_database_manager.h
index 80f0df50be1b2ac507e4f0d119284521327a3e22..a6f3b826fcc0b1e9b42cb832c3d22dcd983202d4 100644
--- a/components/safe_browsing_db/v4_local_database_manager.h
+++ b/components/safe_browsing_db/v4_local_database_manager.h
@@ -58,6 +58,10 @@ class V4LocalDatabaseManager : public SafeBrowsingDatabaseManager {
bool IsMalwareKillSwitchOn() override;
bool IsCsdWhitelistKillSwitchOn() override;
+ protected:
+ std::unordered_set<UpdateListIdentifier> GetStoresForFullHashRequests()
+ override;
+
private:
friend class V4LocalDatabaseManagerTest;
void SetTaskRunnerForTest(
@@ -97,6 +101,10 @@ class V4LocalDatabaseManager : public SafeBrowsingDatabaseManager {
// It should come through the database, from the various V4Stores.
base::hash_map<UpdateListIdentifier, std::string> current_list_states_;
+ // The list of stores to manage (for hash prefixes and full hashes), along
+ // with the corresponding filename on disk for each of them.
+ StoreIdAndFileNames store_id_file_names_;
+
// The protocol manager that downloads the hash prefix updates.
std::unique_ptr<V4UpdateProtocolManager> v4_update_protocol_manager_;

Powered by Google App Engine
This is Rietveld 408576698