[Offline pages] Removing undoing of deleting offline pages

components/offline_pages/offline_page_model_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.h"
 
 #include <stdint.h>
 #include <algorithm>
 #include <utility>
 
@@ skipping 450 matching lines @@
 
   const std::vector<OfflinePageItem>& offline_pages = GetAllPages();
 
   EXPECT_EQ(1UL, offline_pages.size());
   EXPECT_EQ(kTestUrl, offline_pages[0].url);
   EXPECT_EQ(kTestClientId1, offline_pages[0].client_id);
   EXPECT_EQ(kTestFileSize, offline_pages[0].file_size);
   EXPECT_EQ(1, offline_pages[0].access_count);
 }
 
-TEST_F(OfflinePageModelTest, MarkPageForDeletion) {
-  SavePage(kTestUrl, kTestClientId1);
-
-  GURL offline_url = GetAllPages().begin()->GetOfflineURL();
-
-  // Delete the page with undo tiggerred.
-  model()->MarkPageForDeletion(
-      last_save_offline_id(),
-      base::Bind(&OfflinePageModelTest::OnDeletePageDone, AsWeakPtr()));
-  PumpLoop();
-
-  // GetAllPages will not return the page that is marked for deletion.
-  const std::vector<OfflinePageItem>& offline_pages = GetAllPages();
-  EXPECT_EQ(0UL, offline_pages.size());
-
-  EXPECT_FALSE(HasPages(kTestClientNamespace));
-  EXPECT_EQ(nullptr, model()->GetPageByOnlineURL(kTestUrl));
-  EXPECT_EQ(nullptr, model()->GetPageByOfflineId(last_save_offline_id()));
-  EXPECT_EQ(nullptr, model()->GetPageByOfflineURL(offline_url));
-
-  // Undo the deletion.
-  model()->UndoPageDeletion(last_save_offline_id());
-  PumpLoop();
-
-  // GetAllPages will now return the restored page.
-  const std::vector<OfflinePageItem>& offline_pages_after_undo = GetAllPages();
-  EXPECT_EQ(1UL, offline_pages_after_undo.size());
-}
-
-TEST_F(OfflinePageModelTest, FinalizePageDeletion) {
-  scoped_ptr<OfflinePageTestArchiver> archiver(BuildArchiver(
-      kTestUrl, OfflinePageArchiver::ArchiverResult::SUCCESSFULLY_CREATED));
-  model()->SavePage(
-      kTestUrl, kTestClientId1, std::move(archiver),
-      base::Bind(&OfflinePageModelTest::OnSavePageDone, AsWeakPtr()));
-  PumpLoop();
-
-  // Mark the page for deletion.
-  model()->MarkPageForDeletion(
-      last_save_offline_id(),
-      base::Bind(&OfflinePageModelTest::OnDeletePageDone, AsWeakPtr()));
-  PumpLoop();
-
-  EXPECT_EQ(1UL, GetStore()->GetAllPages().size());
-
-  // Fast forward to trigger the page deletion.
-  FastForwardBy(OfflinePageModel::GetFinalDeletionDelayForTesting());
-
-  EXPECT_EQ(0UL, GetStore()->GetAllPages().size());
-}
-
-TEST_F(OfflinePageModelTest, SavePageAfterMarkingPageForDeletion) {
-  scoped_ptr<OfflinePageTestArchiver> archiver(BuildArchiver(
-      kTestUrl, OfflinePageArchiver::ArchiverResult::SUCCESSFULLY_CREATED));
-  model()->SavePage(
-      kTestUrl, kTestClientId1, std::move(archiver),
-      base::Bind(&OfflinePageModelTest::OnSavePageDone, AsWeakPtr()));
-  PumpLoop();
-
-  // Mark the page for deletion.
-  model()->MarkPageForDeletion(
-      last_save_offline_id(),
-      base::Bind(&OfflinePageModelTest::OnDeletePageDone, AsWeakPtr()));
-  PumpLoop();
-
-  EXPECT_EQ(1UL, GetStore()->GetAllPages().size());
-
-  // Re-save the same page.
-  scoped_ptr<OfflinePageTestArchiver> archiver2(BuildArchiver(
-      kTestUrl, OfflinePageArchiver::ArchiverResult::SUCCESSFULLY_CREATED));
-  model()->SavePage(
-      kTestUrl, kTestClientId1, std::move(archiver2),
-      base::Bind(&OfflinePageModelTest::OnSavePageDone, AsWeakPtr()));
-
-  // Fast forward to trigger the page cleanup.
-  FastForwardBy(OfflinePageModel::GetFinalDeletionDelayForTesting());
-
-  // The re-saved page should still exist.
-  const std::vector<OfflinePageItem>& offline_pages = GetAllPages();
-  ASSERT_EQ(1UL, offline_pages.size());
-  EXPECT_EQ(kTestUrl, offline_pages[0].url);
-  EXPECT_EQ(kTestClientId1, offline_pages[0].client_id);
-  EXPECT_EQ(kTestFileSize, offline_pages[0].file_size);
-  EXPECT_EQ(0, offline_pages[0].access_count);
-}
-
 TEST_F(OfflinePageModelTest, GetAllPagesStoreEmpty) {
   const std::vector<OfflinePageItem>& offline_pages = GetAllPages();
 
   EXPECT_EQ(0UL, offline_pages.size());
 }
 
 TEST_F(OfflinePageModelTest, GetAllPagesStoreFailure) {
   GetStore()->set_test_scenario(
       OfflinePageTestStore::TestScenario::LOAD_FAILED);
   const std::vector<OfflinePageItem>& offline_pages = GetAllPages();
@@ skipping 396 matching lines @@
   // Check if feature is correctly enabled by command-line flag.
   base::FeatureList::ClearInstanceForTesting();
   scoped_ptr<base::FeatureList> feature_list(new base::FeatureList);
   feature_list->InitializeFromCommandLine(
       offline_pages::kOfflinePagesBackgroundLoadingFeature.name, "");
   base::FeatureList::SetInstance(std::move(feature_list));
   EXPECT_TRUE(offline_pages::IsOfflinePagesBackgroundLoadingEnabled());
 }
 
 }  // namespace offline_pages