Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(582)

Unified Diff: components/offline_pages/offline_page_model_impl_unittest.cc

Issue 2284933002: Remove OfflineURL from offline page (Closed)
Patch Set: Fix trybot Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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);

Powered by Google App Engine
This is Rietveld 408576698