[Offline pages] Detecting missing offline copy

chrome/browser/android/offline_pages/offline_page_bridge.cc

Index: chrome/browser/android/offline_pages/offline_page_bridge.cc
diff --git a/chrome/browser/android/offline_pages/offline_page_bridge.cc b/chrome/browser/android/offline_pages/offline_page_bridge.cc
index d621b06f8e699aad7f77ec6d2efe8b43a3c86643..089a03fee47a57b0cb3aaadafcc8538109fbbbb2 100644
--- a/chrome/browser/android/offline_pages/offline_page_bridge.cc
+++ b/chrome/browser/android/offline_pages/offline_page_bridge.cc
@@ -103,6 +103,16 @@ void OfflinePageBridge::OfflinePageModelChanged(OfflinePageModel* model) {
   Java_OfflinePageBridge_offlinePageModelChanged(env, obj.obj());
 }
 
+void OfflinePageBridge::OfflinePageDeleted(int64 bookmark_id) {
+  if (!offline_page_model_->is_loaded())

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

Is this likely?

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

Good catch.

+    return;
+  JNIEnv* env = base::android::AttachCurrentThread();
+  ScopedJavaLocalRef<jobject> obj = weak_java_ref_.get(env);
+  if (obj.is_null())
+    return;
+  Java_OfflinePageBridge_offlinePageDeleted(env, obj.obj(), bookmark_id);
+}
+
 void OfflinePageBridge::GetAllPages(JNIEnv* env,
                                     jobject obj,
                                     jobject j_result_obj) {
@@ -201,6 +211,10 @@ void OfflinePageBridge::DeletePages(JNIEnv* env,
       base::Bind(&DeletePageCallback, j_callback_ref));
 }
 
+void OfflinePageBridge::CheckMetadataConsistency(JNIEnv* env, jobject obj) {
+  offline_page_model_->CheckForExternalFileDeletion();
+}
+
 void OfflinePageBridge::NotifyIfDoneLoading() const {
   if (!offline_page_model_->is_loaded())
     return;