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

Unified Diff: components/offline_pages/offline_page_model.h

Issue 1988973002: [Offline pages] Moving disk size related calls to Archive Manager, reorganizing UMA (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@archive-manager
Patch Set: Adding test, updating ArchiveManager implementation to task runner. Created 4 years, 7 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/offline_pages/offline_page_model.h
diff --git a/components/offline_pages/offline_page_model.h b/components/offline_pages/offline_page_model.h
index d03168eed701f7021aa233d000e2595fbddbd36f..0c54543bc06dfc0907282c8d882941c2c66fa700 100644
--- a/components/offline_pages/offline_page_model.h
+++ b/components/offline_pages/offline_page_model.h
@@ -213,17 +213,6 @@ class OfflinePageModel : public KeyedService,
// observers.
void CheckForExternalFileDeletion();
- // Reports the storage histograms related to total size of all stored offline
- // pages. Method is to be called after a page was saved or some pages are
- // deleted. In the latter case |reporting_after_delete| is set to true.
- // Caller is supposed to provide the current |total_space_bytes| on drive
- // where the pages are stored, as well as |free_space_bytes| after the
- // operation was taken. The method will report total size of all pages, and
- // percentage of size of pages as compared to total space and free space.
- void RecordStorageHistograms(int64_t total_space_bytes,
- int64_t free_space_bytes,
- bool reporting_after_delete);
-
// Returns the policy controller.
ClientPolicyController* GetPolicyController();

Powered by Google App Engine
This is Rietveld 408576698