[Offline Pages] Updated clearing logic in storage manager.

components/offline_pages/offline_page_model.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_OFFLINE_PAGES_OFFLINE_PAGE_MODEL_H_
 #define COMPONENTS_OFFLINE_PAGES_OFFLINE_PAGE_MODEL_H_
 
 #include <stdint.h>
 
 #include <map>
@@ skipping 11 matching lines @@
 #include "base/observer_list.h"
 #include "base/optional.h"
 #include "base/supports_user_data.h"
 #include "components/keyed_service/core/keyed_service.h"
 #include "components/offline_pages/offline_page_archiver.h"
 #include "components/offline_pages/offline_page_metadata_store.h"
 #include "components/offline_pages/offline_page_storage_manager.h"
 #include "components/offline_pages/offline_page_types.h"
 
 class GURL;
+class PrefService;
 namespace base {
 class SequencedTaskRunner;
 class Time;
 class TimeDelta;
 class TimeTicks;
 }
 
 namespace offline_pages {
 
 static const char* const kBookmarkNamespace = "bookmark";
@@ skipping 60 matching lines @@
   static int64_t GenerateOfflineId();
 
   // Returns true if an offline copy can be saved for the given URL.
   static bool CanSavePage(const GURL& url);
 
   static base::TimeDelta GetFinalDeletionDelayForTesting();
 
   // 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);
   void RemoveObserver(Observer* observer);
 
   // Attempts to save a page addressed by |url| offline. Requires that the model
   // is loaded.  Generates a new offline id and returns it.
   void SavePage(const GURL& url,
                 const ClientId& client_id,
                 std::unique_ptr<OfflinePageArchiver> archiver,
@@ skipping 237 matching lines @@
   // Delayed tasks that should be invoked after the loading is done.
   std::vector<base::Closure> delayed_tasks_;
 
   // Controller of the client policies.
   std::unique_ptr<ClientPolicyController> policy_controller_;
 
   // Manager for the storage consumed by archives and responsible for
   // 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);
 };
 
 }  // namespace offline_pages
 
 #endif  // COMPONENTS_OFFLINE_PAGES_OFFLINE_PAGE_MODEL_H_