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

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: Addressing feedback 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 b7575a88d269824d486d772e83fd8a04217b177d..eab3d5cecf347787d35782bb08af69da1ba3bf10 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();
« no previous file with comments | « components/offline_pages/archive_manager_unittest.cc ('k') | components/offline_pages/offline_page_model.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698