OfflinePageModel cleanups - improved CanSavePage() and moved static kInvalidOfflineId to impl

components/offline_pages/offline_page_model_impl_unittest.cc

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 2c5a437fc950e469a08fc2246b9554ad5bb5f83c..8459f39357e3efa496d2947b366c7b1f2b63030b 100644
--- a/components/offline_pages/offline_page_model_impl_unittest.cc
+++ b/components/offline_pages/offline_page_model_impl_unittest.cc
@@ -882,15 +882,6 @@ TEST_F(OfflinePageModelImplTest, CheckPagesExistOffline) {
   EXPECT_EQ(existing_pages.end(), existing_pages.find(kTestUrl3));
 }
 
-TEST_F(OfflinePageModelImplTest, CanSavePage) {
-  EXPECT_TRUE(OfflinePageModel::CanSavePage(GURL("http://foo")));
-  EXPECT_TRUE(OfflinePageModel::CanSavePage(GURL("https://foo")));
-  EXPECT_FALSE(OfflinePageModel::CanSavePage(GURL("file:///foo")));
-  EXPECT_FALSE(OfflinePageModel::CanSavePage(GURL("data:image/png;base64,ab")));
-  EXPECT_FALSE(OfflinePageModel::CanSavePage(GURL("chrome://version")));
-  EXPECT_FALSE(OfflinePageModel::CanSavePage(GURL("chrome-native://newtab/")));
-}
-
 TEST_F(OfflinePageModelImplTest, ClearAll) {
   SavePage(kTestUrl, kTestClientId1);
   SavePage(kTestUrl2, kTestClientId2);