Update access info when an offline page is being visited

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 af4ba46b51d4ef26976d44803ab3e84452c0fe8f..a22ea8f8aa22da3abb453677161df590c81589f4 100644
--- a/chrome/browser/android/offline_pages/offline_page_bridge.cc
+++ b/chrome/browser/android/offline_pages/offline_page_bridge.cc
@@ -44,6 +44,17 @@ void SavePageCallback(const ScopedJavaGlobalRef<jobject>& j_callback_obj,
       ConvertUTF8ToJavaString(env, url.spec()).obj());
 }
 
+void MarkPageAccessedCallback(
+    const ScopedJavaGlobalRef<jobject>& j_callback_obj, bool sucecss) {

[newt (away), 2015/09/21 23:02:05]

"success" typo

[jianli, 2015/09/23 00:11:11]

This callback is not longer needed.

+  if (j_callback_obj.is_null())
+    return;
+
+  JNIEnv* env = base::android::AttachCurrentThread();
+
+  Java_MarkPageAccessedCallback_onMarkPageAccessedDone(
+      env, j_callback_obj.obj(), sucecss);
+}
+
 void DeletePageCallback(const ScopedJavaGlobalRef<jobject>& j_callback_obj,
                         OfflinePageModel::DeletePageResult result) {
   JNIEnv* env = base::android::AttachCurrentThread();
@@ -61,7 +72,8 @@ void ToJavaOfflinePageList(JNIEnv* env,
         ConvertUTF8ToJavaString(env, offline_page.url.spec()).obj(),
         offline_page.bookmark_id,
         ConvertUTF8ToJavaString(env, offline_page.GetOfflineURL().spec()).obj(),
-        offline_page.file_size);
+        offline_page.file_size,
+        offline_page.access_count);
   }
 }
 
@@ -126,7 +138,8 @@ ScopedJavaLocalRef<jobject> OfflinePageBridge::GetPageByBookmarkId(
       env, ConvertUTF8ToJavaString(env, offline_page->url.spec()).obj(),
       offline_page->bookmark_id,
       ConvertUTF8ToJavaString(env, offline_page->GetOfflineURL().spec()).obj(),
-      offline_page->file_size);
+      offline_page->file_size,
+      offline_page->access_count);
 }
 
 void OfflinePageBridge::SavePage(JNIEnv* env,
@@ -153,6 +166,19 @@ void OfflinePageBridge::SavePage(JNIEnv* env,
       base::Bind(&SavePageCallback, j_callback_ref, url));
 }
 
+void OfflinePageBridge::MarkPageAccessed(JNIEnv* env,
+                                         jobject obj,
+                                         jobject j_callback_obj,
+                                         jlong bookmark_id) {
+  DCHECK(j_callback_obj);
+
+  ScopedJavaGlobalRef<jobject> j_callback_ref;
+  j_callback_ref.Reset(env, j_callback_obj);
+
+  offline_page_model_->MarkPageAccessed(
+      bookmark_id, base::Bind(&MarkPageAccessedCallback, j_callback_ref));
+}
+
 void OfflinePageBridge::DeletePage(JNIEnv* env,
                                    jobject obj,
                                    jobject j_callback_obj,