[Offline pages] Detecting missing offline copy

chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridge.java

Index: chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridge.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridge.java b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridge.java
index 0a5a923041bcbe99a4a873e43979c3d7b6bd66d5..5541b339bcd24e52ee9dd482c71bd2d3aaa11d3c 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridge.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridge.java
@@ -15,6 +15,7 @@ import org.chromium.base.metrics.RecordHistogram;
 import org.chromium.chrome.browser.BookmarksBridge;
 import org.chromium.chrome.browser.profiles.Profile;
 import org.chromium.components.bookmarks.BookmarkId;
+import org.chromium.components.bookmarks.BookmarkType;
 import org.chromium.components.offlinepages.SavePageResult;
 import org.chromium.content.browser.ContentViewCore;
 import org.chromium.content_public.browser.WebContents;
@@ -69,8 +70,7 @@ public final class OfflinePageBridge {
     }
 
     /**
-     * Base empty implementation observer class that provides listeners to be notified of changes to
-     * the offline page model.
+     *  Base observer class listeners to be notified of changes to the offline page model.

[jianli, 2015/10/20 21:56:48]

remove the extra space before 'B'

[fgorski, 2015/10/21 20:01:33]

Done.

      */
     public abstract static class OfflinePageModelObserver {
         /**
@@ -83,6 +83,12 @@ public final class OfflinePageBridge {
          * update an offline page.
          */
         public void offlinePageModelChanged() {}
+
+        /**
+         * Called when an offline page is deleted.

[Ian Wen, 2015/10/20 20:52:01]

Document "This can be called as a result of #checkOfflinePageMetaData()"

[fgorski, 2015/10/21 20:01:33]

Done.

+         * @param bookmarkId A bookmark ID of the deleted offline page.
+         */
+        public void offlinePageDeleted(BookmarkId bookmarkId) {}
     }
 
     private static int getFreeSpacePercentage() {
@@ -267,6 +273,13 @@ public final class OfflinePageBridge {
         return result;
     }
 
+    /**
+     * Starts a check of offline page metadata, e.g. are all offline copies present.
+     */
+    public void checkOfflinePageMetadata() {
+        nativeCheckMetadataConsistency(mNativeOfflinePageBridge);
+    }
+
     @CalledByNative
     private void offlinePageModelLoaded() {
         mIsNativeOfflinePageModelLoaded = true;
@@ -283,6 +296,14 @@ public final class OfflinePageBridge {
     }
 
     @CalledByNative
+    private void offlinePageDeleted(long bookmarkId) {
+        BookmarkId id = new BookmarkId(bookmarkId, BookmarkType.NORMAL);
+        for (OfflinePageModelObserver observer : mObservers) {
+            observer.offlinePageDeleted(id);
+        }
+    }
+
+    @CalledByNative
     private static void createOfflinePageAndAddToList(List<OfflinePageItem> offlinePagesList,
             String url, long bookmarkId, String offlineUrl, long fileSize, int accessCount) {
         offlinePagesList.add(createOfflinePageItem(url, bookmarkId, offlineUrl, fileSize,
@@ -312,4 +333,5 @@ public final class OfflinePageBridge {
             long nativeOfflinePageBridge, DeletePageCallback callback, long[] bookmarkIds);
     private native void nativeGetPagesToCleanUp(
             long nativeOfflinePageBridge, List<OfflinePageItem> offlinePages);
+    private native void nativeCheckMetadataConsistency(long nativeOfflinePageBridge);
 }