Update access info when an offline page is being visited

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 26 matching lines @@
 void SavePageCallback(const ScopedJavaGlobalRef<jobject>& j_callback_obj,
                       const GURL& url,
                       OfflinePageModel::SavePageResult result) {
   JNIEnv* env = base::android::AttachCurrentThread();
 
   Java_SavePageCallback_onSavePageDone(
       env, j_callback_obj.obj(), static_cast<int>(result),
       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();
 
   Java_DeletePageCallback_onDeletePageDone(
       env, j_callback_obj.obj(), static_cast<int>(result));
 }
 
 void ToJavaOfflinePageList(JNIEnv* env,
                            jobject j_result_obj,
                            const std::vector<OfflinePageItem>& offline_pages) {
   for (const OfflinePageItem& offline_page : offline_pages) {
     Java_OfflinePageBridge_createOfflinePageAndAddToList(
         env, j_result_obj,
         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);
   }
 }
 
 }  // namespace
 
 static jboolean IsOfflinePagesEnabled(JNIEnv* env,
                                       const JavaParamRef<jclass>& clazz) {
   return offline_pages::IsOfflinePagesEnabled();
 }
 
@@ skipping 44 matching lines @@
     jlong bookmark_id) {
   const OfflinePageItem* offline_page =
       offline_page_model_->GetPageByBookmarkId(bookmark_id);
   if (!offline_page)
     return ScopedJavaLocalRef<jobject>();
 
   return Java_OfflinePageBridge_createOfflinePageItem(
       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,
                                  jobject obj,
                                  jobject j_callback_obj,
                                  jobject j_web_contents,
                                  jlong bookmark_id) {
   DCHECK(j_callback_obj);
   DCHECK(j_web_contents);
 
   ScopedJavaGlobalRef<jobject> j_callback_ref;
   j_callback_ref.Reset(env, j_callback_obj);
 
   content::WebContents* web_contents =
       content::WebContents::FromJavaWebContents(j_web_contents);
   GURL url(web_contents->GetLastCommittedURL());
 
   scoped_ptr<OfflinePageArchiver> archiver(
       new OfflinePageMHTMLArchiver(
           web_contents, GetDownloadsPath(browser_context_)));
 
   offline_page_model_->SavePage(
       url, bookmark_id, archiver.Pass(),
       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,
                                    jlong bookmark_id) {
   DCHECK(j_callback_obj);
 
   ScopedJavaGlobalRef<jobject> j_callback_ref;
   j_callback_ref.Reset(env, j_callback_obj);
 
   offline_page_model_->DeletePageByBookmarkId(bookmark_id, base::Bind(
@@ skipping 57 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