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..3652c6004ebb732e0cb86a811788928d0f1fc14c 100644
--- a/components/offline_pages/offline_page_model_impl_unittest.cc
+++ b/components/offline_pages/offline_page_model_impl_unittest.cc
@@ -882,13 +882,14 @@ 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, CanSaveURL) {
+  EXPECT_TRUE(OfflinePageModel::CanSaveURL(GURL("http://foo")));
+  EXPECT_TRUE(OfflinePageModel::CanSaveURL(GURL("https://foo")));
+  EXPECT_FALSE(OfflinePageModel::CanSaveURL(GURL("file:///foo")));
+  EXPECT_FALSE(OfflinePageModel::CanSaveURL(GURL("data:image/png;base64,ab")));
+  EXPECT_FALSE(OfflinePageModel::CanSaveURL(GURL("chrome://version")));
+  EXPECT_FALSE(OfflinePageModel::CanSaveURL(GURL("chrome-native://newtab/")));
+  EXPECT_FALSE(OfflinePageModel::CanSaveURL(GURL("/invalid/url.mhtml")));
 }
 
 TEST_F(OfflinePageModelImplTest, ClearAll) {