| Index: components/offline_pages/offline_page_model_impl_unittest.cc
|
| diff --git a/components/offline_pages/offline_page_model_impl_unittest.cc b/components/offline_pages/offline_page_model_impl_unittest.cc
|
| index 39dda8e40b89f08591f23fb9dcbdef64a16b061a..7021ebb3a9427064767f6aa0847079b1f49f7669 100644
|
| --- a/components/offline_pages/offline_page_model_impl_unittest.cc
|
| +++ b/components/offline_pages/offline_page_model_impl_unittest.cc
|
| @@ -140,9 +140,7 @@ class OfflinePageModelImplTest
|
|
|
| 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(); }
|
|
|
| @@ -375,17 +373,6 @@ const OfflinePageItem* OfflinePageModelImplTest::GetPageByOfflineId(
|
| 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) {
|
| @@ -889,34 +876,6 @@ TEST_F(OfflinePageModelImplTest, GetPageByOfflineId) {
|
| 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);
|
|
|