Wipe out offline page data on clearing cookie and site data

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 <map>
 #include <vector>
 
@@ skipping 149 matching lines @@
   // Deletes an offline page related to the passed |bookmark_id|. Requires that
   // the model is loaded.
   void DeletePageByBookmarkId(int64 bookmark_id,
                               const DeletePageCallback& callback);
 
   // Deletes offline pages related to the passed |bookmark_ids|. Requires that
   // the model is loaded.
   void DeletePagesByBookmarkId(const std::vector<int64>& bookmark_ids,
                                const DeletePageCallback& callback);
 
-  void UndeletePage(int64 bookmark_id, const DeletePageCallback& callback);
+  // Remove all saved files and clear the store.
+  void ClearAll(const base::Closure& callback);

[fgorski, 2015/10/23 20:55:03]

DeleteAllPages -- to be consistent with the naming above.

[jianli, 2015/10/26 21:42:44]

I think semantically ClearAll is a bit different from DeleteAllPages since the
former also means to destroy the whole store while the latter does not need
this. I rather use ClearAll to mean a special wipe out like this.

[fgorski, 2015/10/27 20:13:09]

Acknowledged.

 
   // Gets all available offline pages. Requires that the model is loaded.
   const std::vector<OfflinePageItem> GetAllPages() const;
 
   // Gets pages that should be removed to clean up storage. Requires that the
   // model is loaded.
   const std::vector<OfflinePageItem> GetPagesToCleanUp() const;
 
   // Returns an offline page associated with a specified |bookmark_id|. nullptr
   // is returned if not found.
@@ skipping 75 matching lines @@
   void UndoPageDeletion(int64 bookmark_id);
   void OnUndoOfflinePageDone(const OfflinePageItem& offline_page, bool success);
 
   // Callbacks for checking if offline pages are missing archive files.
   void OnFindPagesMissingArchiveFile(
       const std::vector<int64>* pages_missing_archive_file);
   void OnRemoveOfflinePagesMissingArchiveFileDone(
       const std::vector<int64>& bookmark_ids,
       OfflinePageModel::DeletePageResult result);
 
+  // Steps for clearing all.
+  void OnRemoveAllFilesDoneForClearAll(const base::Closure& callback,
+                                       DeletePageResult result);
+  void OnResetStoreDoneForClearAll(const base::Closure& callback, bool success);
+  void OnReloadStoreDoneForClearAll(
+      const base::Closure& callback,
+      bool success,
+      const std::vector<OfflinePageItem>& offline_pages);
+
+  void CacheLoadedData(const std::vector<OfflinePageItem>& offline_pages);
+
   // Persistent store for offline page metadata.
   scoped_ptr<OfflinePageMetadataStore> store_;
 
   // The observers.
   base::ObserverList<Observer> observers_;
 
   bool is_loaded_;
 
   // In memory copy of the offline page metadata, keyed by bookmark IDs.
   std::map<int64, OfflinePageItem> offline_pages_;
@@ skipping 10 matching lines @@
       scoped_observer_;
 
   base::WeakPtrFactory<OfflinePageModel> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(OfflinePageModel);
 };
 
 }  // namespace offline_pages
 
 #endif  // COMPONENTS_OFFLINE_PAGES_OFFLINE_PAGE_MODEL_H_