[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..727431444b2129faa5e0df9f4bcbcae9fe2936c0 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,30 @@ 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));
+  ResetModel();

[fgorski, 2016/06/08 20:27:07]

If there is implicit stuff going on, can you add a comment?

[romax, 2016/06/08 22:18:59]

Done.

+  PumpLoop();
+
+  EXPECT_EQ(0UL, GetAllPages().size());
+  EXPECT_FALSE(base::PathExists(path));
+}
+
 TEST_F(OfflinePageModelImplTest, DeleteMultiplePages) {
   OfflinePageTestStore* store = GetStore();