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

Unified Diff: components/offline_pages/offline_page_storage_manager.h

Issue 2509403002: Move const double data out of header file (Closed)
Patch Set: Created 4 years, 1 month 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_storage_manager.h
diff --git a/components/offline_pages/offline_page_storage_manager.h b/components/offline_pages/offline_page_storage_manager.h
index 28fa130002123d0122577f9485ba15f8ece573ec..914212d2cd22e41e1b5a25833a76cd62c51e8fe2 100644
--- a/components/offline_pages/offline_page_storage_manager.h
+++ b/components/offline_pages/offline_page_storage_manager.h
@@ -24,14 +24,19 @@ namespace offline_pages {
// Maximum % of total available storage that will be occupied by offline pages
// before a storage clearup.
-const double kOfflinePageStorageLimit = 0.3;
-// The target % of storage usage we try to reach below when expiring pages.
-const double kOfflinePageStorageClearThreshold = 0.1;
-// The time that the storage cleanup will be triggered again since the last one.
-const base::TimeDelta kClearStorageInterval = base::TimeDelta::FromMinutes(10);
-// The time that the page record will be removed from the store since the page
-// has been expired.
-const base::TimeDelta kRemovePageItemInterval = base::TimeDelta::FromDays(21);
+struct constants {
+ static constexpr double kOfflinePageStorageLimit = 0.3;
+ // The target % of storage usage we try to reach below when expiring pages.
+ static constexpr double kOfflinePageStorageClearThreshold = 0.1;
+ // The time that the storage cleanup will be triggered again since the last
+ // one.
+ static constexpr base::TimeDelta kClearStorageInterval =
+ base::TimeDelta::FromMinutes(10);
+ // The time that the page record will be removed from the store since the page
+ // has been expired.
+ static constexpr base::TimeDelta kRemovePageItemInterval =
+ base::TimeDelta::FromDays(21);
+};
class ClientPolicyController;
class OfflinePageModel;

Powered by Google App Engine
This is Rietveld 408576698