[Offline pages] Properly informing observers when offline page is deleted

components/offline_pages/offline_page_model.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/offline_pages/offline_page_model.h"
 
 #include <algorithm>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 551 matching lines @@
         "OfflinePages.DeletePage.LastOpenToCreated",
         (iter->second.last_access_time - iter->second.creation_time).
             InMinutes(),
         1,
         base::TimeDelta::FromDays(365).InMinutes(),
         100);
     UMA_HISTOGRAM_MEMORY_KB(
         "OfflinePages.DeletePage.PageSize", iter->second.file_size / 1024);
     UMA_HISTOGRAM_COUNTS(
         "OfflinePages.DeletePage.AccessCount", iter->second.access_count);
+    // If the page is not marked for deletion at this point, the model has not
+    // yet informed the observer that the offline page is deleted.
+    if (!iter->second.IsMarkedForDeletion()) {
+      FOR_EACH_OBSERVER(Observer, observers_,
+                        OfflinePageDeleted(iter->second.bookmark_id));
+    }
     offline_pages_.erase(iter);
   }
   if (bookmark_ids.size() > 1) {
     UMA_HISTOGRAM_COUNTS(
         "OfflinePages.BatchDelete.Count", bookmark_ids.size());
     UMA_HISTOGRAM_MEMORY_KB(
         "OfflinePages.BatchDelete.TotalPageSize", total_size / 1024);
   }
   // Deleting multiple pages always succeeds when it gets to this point.
   InformDeletePageDone(
@@ skipping 76 matching lines @@
 }
 
 void OfflinePageModel::CacheLoadedData(
     const std::vector<OfflinePageItem>& offline_pages) {
   offline_pages_.clear();
   for (const auto& offline_page : offline_pages)
     offline_pages_[offline_page.bookmark_id] = offline_page;
 }
 
 }  // namespace offline_pages