Remove OfflineURL from offline page

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 122 matching lines @@
   }
 
   bool HasPages(std::string name_space);
 
   CheckPagesExistOfflineResult CheckPagesExistOffline(std::set<GURL>);
 
   MultipleOfflineIdResult GetOfflineIdsForClientId(const ClientId& client_id);
 
   const OfflinePageItem* GetPageByOfflineId(int64_t offline_id);
 
-  MultipleOfflinePageItemResult GetPagesByOnlineURL(const GURL& offline_url);
-
-  const OfflinePageItem* GetPageByOfflineURL(const GURL& offline_url);
+  MultipleOfflinePageItemResult GetPagesByOnlineURL(const GURL& online_url);
 
   OfflinePageModelImpl* model() { return model_.get(); }
 
   int64_t last_save_offline_id() const { return last_save_offline_id_; }
 
   SavePageResult last_save_result() const { return last_save_result_; }
 
   DeletePageResult last_delete_result() const { return last_delete_result_; }
 
   int64_t last_deleted_offline_id() const { return last_deleted_offline_id_; }
@@ skipping 212 matching lines @@
     int64_t offline_id) {
   const OfflinePageItem* result = nullptr;
   model()->GetPageByOfflineId(
       offline_id,
       base::Bind(&OfflinePageModelImplTest::OnGetSingleOfflinePageItemResult,
                  AsWeakPtr(), base::Unretained(&result)));
   PumpLoop();
   return result;
 }
 
-const OfflinePageItem* OfflinePageModelImplTest::GetPageByOfflineURL(
-    const GURL& offline_url) {
-  const OfflinePageItem* result = nullptr;
-  model()->GetPageByOfflineURL(
-      offline_url,
-      base::Bind(&OfflinePageModelImplTest::OnGetSingleOfflinePageItemResult,
-                 AsWeakPtr(), base::Unretained(&result)));
-  PumpLoop();
-  return result;
-}
-
 void OfflinePageModelImplTest::OnGetSingleOfflinePageItemResult(
     const OfflinePageItem** storage,
     const OfflinePageItem* result) {
   *storage = result;
 }
 
 void OfflinePageModelImplTest::OnGetMultipleOfflinePageItemsResult(
     MultipleOfflinePageItemResult* storage,
     const MultipleOfflinePageItemResult& result) {
   *storage = result;
@@ skipping 483 matching lines @@
   page = GetPageByOfflineId(offline2);
   EXPECT_TRUE(page);
   EXPECT_EQ(kTestUrl2, page->url);
   EXPECT_EQ(kTestClientId2, page->client_id);
   EXPECT_EQ(kTestFileSize, page->file_size);
 
   page = GetPageByOfflineId(-42);
   EXPECT_FALSE(page);
 }
 
-TEST_F(OfflinePageModelImplTest, GetPageByOfflineURL) {
-  SavePage(kTestUrl, kTestClientId1);
-  int64_t offline1 = last_save_offline_id();
-
-  OfflinePageTestStore* store = GetStore();
-  GURL offline_url = store->last_saved_page().GetOfflineURL();
-
-  SavePage(kTestUrl2, kTestClientId2);
-
-  GURL offline_url2 = store->last_saved_page().GetOfflineURL();
-  int64_t offline2 = last_save_offline_id();
-
-  const OfflinePageItem* page = GetPageByOfflineURL(offline_url2);
-  EXPECT_TRUE(page);
-  EXPECT_EQ(kTestUrl2, page->url);
-  EXPECT_EQ(kTestClientId2, page->client_id);
-  EXPECT_EQ(offline2, page->offline_id);
-
-  page = GetPageByOfflineURL(offline_url);
-  EXPECT_TRUE(page);
-  EXPECT_EQ(kTestUrl, page->url);
-  EXPECT_EQ(kTestClientId1, page->client_id);
-  EXPECT_EQ(offline1, page->offline_id);
-
-  page = GetPageByOfflineURL(GURL("http://foo"));
-  EXPECT_FALSE(page);
-}
-
 TEST_F(OfflinePageModelImplTest, GetPagesByOnlineURL) {
   SavePage(kTestUrl, kTestClientId1);
   SavePage(kTestUrl2, kTestClientId2);
 
   MultipleOfflinePageItemResult pages = GetPagesByOnlineURL(kTestUrl2);
   EXPECT_EQ(1U, pages.size());
   EXPECT_EQ(kTestUrl2, pages[0].url);
   EXPECT_EQ(kTestClientId2, pages[0].client_id);
 
   pages = GetPagesByOnlineURL(kTestUrl);
@@ skipping 257 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