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

Unified Diff: components/safe_browsing_db/v4_local_database_manager.cc

Issue 2447443002: Log the size of each of the stores and complete DB on launch and after each (Closed)
Patch Set: shess@'s review 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.cc
diff --git a/components/safe_browsing_db/v4_local_database_manager.cc b/components/safe_browsing_db/v4_local_database_manager.cc
index 5798bf2af85f7031247620061388d09ce9695088..bef4313f5441acd25ce54fd984a2cdae5e3b7cbd 100644
--- a/components/safe_browsing_db/v4_local_database_manager.cc
+++ b/components/safe_browsing_db/v4_local_database_manager.cc
@@ -296,6 +296,8 @@ void V4LocalDatabaseManager::DatabaseReadyForChecks(
if (enabled_) {
v4_database_ = std::move(v4_database);
+ v4_database_->RecordFileSizeHistograms();
+
// The consistency of the stores read from the disk needs to verified. Post
// that task on the task runner. It calls |DatabaseReadyForUpdates|
// callback with the stores to reset, if any, and then we can schedule the
@@ -324,6 +326,7 @@ void V4LocalDatabaseManager::DatabaseReadyForUpdates(
void V4LocalDatabaseManager::DatabaseUpdated() {
if (enabled_) {
+ v4_database_->RecordFileSizeHistograms();
v4_update_protocol_manager_->ScheduleNextUpdate(
v4_database_->GetStoreStateMap());
}

Powered by Google App Engine
This is Rietveld 408576698