[Offline Pages] Filtering expired pages and fix consistency check.

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 3652c6004ebb732e0cb86a811788928d0f1fc14c..a1b6ba168464f7db13ba04a88c5918455e67d156 100644
--- a/components/offline_pages/offline_page_model_impl_unittest.cc
+++ b/components/offline_pages/offline_page_model_impl_unittest.cc
@@ -727,10 +727,10 @@ TEST_F(OfflinePageModelImplTest, DetectThatOfflineCopyIsMissing) {
 
   // Delete the offline copy of the page and check the metadata.
   base::DeleteFile(page->file_path, false);
-  model()->CheckForExternalFileDeletion();
+  model()->CheckMetadataConsistency();
   PumpLoop();
 
-  EXPECT_EQ(last_deleted_offline_id(), offline_id);
+  // Check if the page has been expired.
   EXPECT_EQ(0UL, GetAllPages().size());
 }
 
@@ -749,10 +749,33 @@ TEST_F(OfflinePageModelImplTest, DetectThatOfflineCopyIsMissingAfterLoad) {
   ResetModel();
   PumpLoop();
 
-  EXPECT_EQ(last_deleted_offline_id(), offline_id);
+  // Check if the page has been expired.
   EXPECT_EQ(0UL, GetAllPages().size());
 }
 
+TEST_F(OfflinePageModelImplTest, DetectThatHeadlessPageIsDeleted) {
+  // Save a page.
+  SavePage(kTestUrl, kTestClientId1);
+  PumpLoop();
+  int64_t offline_id = last_save_offline_id();
+
+  ResetResults();
+  base::Optional<OfflinePageItem> page = GetPageByOfflineId(offline_id);
+  base::FilePath path = page->file_path;
+  EXPECT_TRUE(base::PathExists(path));
+  GetStore()->ClearAllPages();
+
+  EXPECT_TRUE(base::PathExists(path));
+  // Since we've manually changed the store, we have to reload the model to
+  // actually refresh the in-memory copy in model. Otherwise GetAllPages() would
+  // still have the page we saved above.
+  ResetModel();
+  PumpLoop();
+
+  EXPECT_EQ(0UL, GetAllPages().size());
+  EXPECT_FALSE(base::PathExists(path));
+}
+
 TEST_F(OfflinePageModelImplTest, DeleteMultiplePages) {
   OfflinePageTestStore* store = GetStore();