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

Unified Diff: components/safe_browsing_db/v4_local_database_manager.h

Issue 2384893002: PVer4: Test checksum on startup outside the hotpath of DB load (Closed)
Patch Set: Verify that the checksum check happens async Created 4 years, 2 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 c73db1b97164ae41238b5861db9cb214b0ce7bc3..6b9a997226d87cb4ea9fa702e746cb1abe0a3e07 100644
--- a/components/safe_browsing_db/v4_local_database_manager.h
+++ b/components/safe_browsing_db/v4_local_database_manager.h
@@ -134,8 +134,15 @@ class V4LocalDatabaseManager : public SafeBrowsingDatabaseManager {
~V4LocalDatabaseManager() override;
// Called when all the stores managed by the database have been read from
- // disk after startup and the database is ready for use.
- void DatabaseReady(std::unique_ptr<V4Database> v4_database);
+ // disk after startup and the database is ready for checking resource
+ // reputation.
+ void DatabaseReadyForChecks(std::unique_ptr<V4Database> v4_database);
+
+ // Called when all the stores managed by the database have been verified for
+ // checksum correctness after startup and the database is ready for applying
+ // updates.
+ void DatabaseReadyForUpdates(
+ const std::vector<ListIdentifier>& stores_to_reset);
// Called when the database has been updated and schedules the next update.
void DatabaseUpdated();
« no previous file with comments | « components/safe_browsing_db/v4_database_unittest.cc ('k') | components/safe_browsing_db/v4_local_database_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698