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

Unified Diff: components/offline_pages/offline_page_model.h

Issue 1986673002: [Offline Pages] Updated clearing logic in storage manager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fixing trybots. 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 5f5ff65ac6ac89ba3820238889f754802b3b0cba..a343318275048ffd39622821678f2f5c50ba8ba9 100644
--- a/components/offline_pages/offline_page_model.h
+++ b/components/offline_pages/offline_page_model.h
@@ -29,6 +29,7 @@
#include "components/offline_pages/offline_page_types.h"
class GURL;
+class PrefService;
namespace base {
class SequencedTaskRunner;
class Time;
@@ -109,7 +110,8 @@ class OfflinePageModel : public KeyedService,
// All blocking calls/disk access will happen on the provided |task_runner|.
OfflinePageModel(std::unique_ptr<OfflinePageMetadataStore> store,
const base::FilePath& archives_dir,
- const scoped_refptr<base::SequencedTaskRunner>& task_runner);
+ const scoped_refptr<base::SequencedTaskRunner>& task_runner,
+ PrefService* prefs);
~OfflinePageModel() override;
void AddObserver(Observer* observer);
@@ -367,6 +369,9 @@ class OfflinePageModel : public KeyedService,
// automatic page clearing.
std::unique_ptr<OfflinePageStorageManager> storage_manager_;
+ // PrefService which should be used by the storage manager.
fgorski 2016/05/17 05:33:29 Not owned.
romax 2016/05/18 02:36:19 removing
+ PrefService* prefs_;
+
base::WeakPtrFactory<OfflinePageModel> weak_ptr_factory_;
DISALLOW_COPY_AND_ASSIGN(OfflinePageModel);

Powered by Google App Engine
This is Rietveld 408576698