Chromium Code Reviews| OLD | NEW |
|---|---|
| (Empty) | |
| 1 // Copyright 2017 The Chromium Authors. All rights reserved. | |
| 2 // Use of this source code is governed by a BSD-style license that can be | |
| 3 // found in the LICENSE file. | |
| 4 | |
| 5 #ifndef COMPONENTS_OFFLINE_PAGES_CORE_CACHED_OFFLINE_PAGE_UTILS_H_ | |
| 6 #define COMPONENTS_OFFLINE_PAGES_CORE_CACHED_OFFLINE_PAGE_UTILS_H_ | |
| 7 | |
| 8 #include "components/offline_pages/core/offline_page_types.h" | |
| 9 | |
| 10 namespace base { | |
| 11 class Time; | |
| 12 } | |
| 13 | |
| 14 namespace offline_pages { | |
| 15 | |
| 16 class OfflinePageModel; | |
| 17 | |
| 18 // A dedicated file of static methods used for all upper level cached offline | |
| 19 // pages related utility functions. For example: | |
| 20 // - Get total size of all cached offline pages. | |
| 21 | |
| 22 // Get total size of cache offline pages for a given time range. | |
| 23 void GetCachedOfflinePageSizeBetween(OfflinePageModel* model, | |
|
dewittj
2017/05/03 21:40:51
It might be better to move this to //components/of
romax
2017/05/04 04:20:36
Done.
I'm adding these into offline_page_utils the
| |
| 24 const SizeCalculatedCallback& callback, | |
| 25 const base::Time& begin_time, | |
|
dewittj
2017/05/03 21:40:51
nit: #include "base/time/time.h"
romax
2017/05/04 04:20:36
I thought a forward declaration would be enough?
| |
| 26 const base::Time& end_time); | |
| 27 | |
| 28 } // namespace offline_pages | |
| 29 #endif // COMPONENTS_OFFLINE_PAGES_CORE_CACHED_OFFLINE_PAGE_UTILS_H_ | |
| OLD | NEW |