[Offline pages] Detecting missing offline copy

chrome/browser/android/offline_pages/offline_page_bridge.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 "chrome/browser/android/offline_pages/offline_page_bridge.h"
 
 #include "base/android/jni_array.h"
 #include "base/android/jni_string.h"
 #include "base/basictypes.h"
 #include "base/files/file_path.h"
@@ skipping 85 matching lines @@
 
 void OfflinePageBridge::OfflinePageModelChanged(OfflinePageModel* model) {
   DCHECK_EQ(offline_page_model_, model);
   JNIEnv* env = base::android::AttachCurrentThread();
   ScopedJavaLocalRef<jobject> obj = weak_java_ref_.get(env);
   if (obj.is_null())
     return;
   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) {
   DCHECK(offline_page_model_->is_loaded());
   DCHECK(j_result_obj);
   const std::vector<OfflinePageItem>& offline_pages =
       offline_page_model_->GetAllPages();
   ToJavaOfflinePageList(env, j_result_obj, offline_pages);
 }
 
@@ skipping 78 matching lines @@
 
   std::vector<int64> bookmark_ids;
   base::android::JavaLongArrayToInt64Vector(env, bookmark_ids_array,
                                             &bookmark_ids);
 
   offline_page_model_->DeletePagesByBookmarkId(
       bookmark_ids,
       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;
   JNIEnv* env = base::android::AttachCurrentThread();
   ScopedJavaLocalRef<jobject> obj = weak_java_ref_.get(env);
   if (obj.is_null())
     return;
   Java_OfflinePageBridge_offlinePageModelLoaded(env, obj.obj());
 }
 
@@ skipping 26 matching lines @@
   return reinterpret_cast<jlong>(new OfflinePageBridge(
       env, obj, ProfileAndroid::FromProfileAndroid(j_profile)));
 }
 
 bool RegisterOfflinePageBridge(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace android
 }  // namespace offline_pages