Offline Pages - Removes the "ClearAllPages" API.

components/offline_pages/offline_page_model_impl_unittest.cc

 // 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.
 
 #include "components/offline_pages/offline_page_model_impl.h"
 
 #include <stdint.h>
 #include <algorithm>
 #include <utility>
 
@@ skipping 66 matching lines @@
   void OfflinePageDeleted(int64_t offline_id,
                           const ClientId& client_id) override;
 
   // OfflinePageTestArchiver::Observer implementation.
   void SetLastPathCreatedByArchiver(const base::FilePath& file_path) override;
 
   // OfflinePageModel callbacks.
   void OnSavePageDone(SavePageResult result, int64_t offline_id);
   void OnDeletePageDone(DeletePageResult result);
   void OnCheckPagesExistOfflineDone(const CheckPagesExistOfflineResult& result);
-  void OnClearAllDone();
   void OnGetOfflineIdsForClientIdDone(MultipleOfflineIdResult* storage,
                                       const MultipleOfflineIdResult& result);
   void OnGetSingleOfflinePageItemResult(const OfflinePageItem** storage,
                                         const OfflinePageItem* result);
   void OnGetMultipleOfflinePageItemsResult(
       MultipleOfflinePageItemResult* storage,
       const MultipleOfflinePageItemResult& result);
   void OnPagesExpired(bool result);
 
   // OfflinePageMetadataStore callbacks.
@@ skipping 141 matching lines @@
 
 void OfflinePageModelImplTest::OnDeletePageDone(DeletePageResult result) {
   last_delete_result_ = result;
 }
 
 void OfflinePageModelImplTest::OnCheckPagesExistOfflineDone(
     const CheckPagesExistOfflineResult& result) {
   last_pages_exist_result_ = result;
 }
 
-void OfflinePageModelImplTest::OnClearAllDone() {
-  PumpLoop();
-}
-
 void OfflinePageModelImplTest::OnStoreUpdateDone(bool /* success - ignored */) {
 }
 
 std::unique_ptr<OfflinePageTestArchiver>
 OfflinePageModelImplTest::BuildArchiver(
     const GURL& url,
     OfflinePageArchiver::ArchiverResult result) {
   return std::unique_ptr<OfflinePageTestArchiver>(
       new OfflinePageTestArchiver(this, url, result, kTestTitle, kTestFileSize,
                                   base::ThreadTaskRunnerHandle::Get()));
@@ skipping 696 matching lines @@
 TEST_F(OfflinePageModelImplTest, CanSaveURL) {
   EXPECT_TRUE(OfflinePageModel::CanSaveURL(GURL("http://foo")));
   EXPECT_TRUE(OfflinePageModel::CanSaveURL(GURL("https://foo")));
   EXPECT_FALSE(OfflinePageModel::CanSaveURL(GURL("file:///foo")));
   EXPECT_FALSE(OfflinePageModel::CanSaveURL(GURL("data:image/png;base64,ab")));
   EXPECT_FALSE(OfflinePageModel::CanSaveURL(GURL("chrome://version")));
   EXPECT_FALSE(OfflinePageModel::CanSaveURL(GURL("chrome-native://newtab/")));
   EXPECT_FALSE(OfflinePageModel::CanSaveURL(GURL("/invalid/url.mhtml")));
 }
 
-TEST_F(OfflinePageModelImplTest, ClearAll) {
-  SavePage(kTestUrl, kTestClientId1);
-  SavePage(kTestUrl2, kTestClientId2);
-
-  const std::vector<OfflinePageItem>& offline_pages = GetAllPages();
-  EXPECT_EQ(2UL, offline_pages.size());
-  EXPECT_EQ(2UL, GetStore()->GetAllPages().size());
-  base::FilePath archiver_path = offline_pages[0].file_path;
-  EXPECT_TRUE(base::PathExists(archiver_path));
-
-  // ClearAll should delete all the files and wipe out both cache and store.
-  model()->ClearAll(
-      base::Bind(&OfflinePageModelImplTest::OnClearAllDone, AsWeakPtr()));
-  PumpLoop();
-  EXPECT_EQ(0UL, GetAllPages().size());
-  EXPECT_EQ(0UL, GetStore()->GetAllPages().size());
-  EXPECT_FALSE(base::PathExists(archiver_path));
-
-  // The model should reload the store after the reset. All model operations
-  // should continue to work.
-  SavePage(kTestUrl2, kTestClientId2);
-  EXPECT_EQ(1UL, GetAllPages().size());
-  EXPECT_EQ(1UL, GetStore()->GetAllPages().size());
-}
-
 TEST_F(OfflinePageModelImplTest, SaveRetrieveMultipleClientIds) {
   EXPECT_FALSE(HasPages(kTestClientNamespace));
   SavePage(kTestUrl, kTestClientId1);
   int64_t offline1 = last_save_offline_id();
   EXPECT_TRUE(HasPages(kTestClientNamespace));
 
   SavePage(kTestUrl2, kTestClientId1);
   int64_t offline2 = last_save_offline_id();
 
   EXPECT_NE(offline1, offline2);
@@ skipping 223 matching lines @@
   // Check if feature is correctly enabled by command-line flag.
   scoped_feature_list.reset(new base::test::ScopedFeatureList);
   scoped_feature_list->InitFromCommandLine(
       std::string(kOfflineBookmarksFeature.name) + "," +
           kOfflinePagesSharingFeature.name,
       "");
   EXPECT_TRUE(offline_pages::IsOfflinePagesSharingEnabled());
 }
 
 }  // namespace offline_pages