Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(618)

Side by Side Diff: chrome/browser/android/offline_pages/offline_page_bridge.cc

Issue 1739163005: Java side of purging BookmarkId from offline pages (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Java side of offline id changes Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « chrome/android/javatests/src/org/chromium/chrome/browser/offlinepages/OfflinePageUtilsTest.java ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/android/offline_pages/offline_page_bridge.h" 5 #include "chrome/browser/android/offline_pages/offline_page_bridge.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/android/jni_array.h" 9 #include "base/android/jni_array.h"
10 #include "base/android/jni_string.h" 10 #include "base/android/jni_string.h"
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
52 } 52 }
53 53
54 void ToJavaOfflinePageList(JNIEnv* env, 54 void ToJavaOfflinePageList(JNIEnv* env,
55 jobject j_result_obj, 55 jobject j_result_obj,
56 const std::vector<OfflinePageItem>& offline_pages) { 56 const std::vector<OfflinePageItem>& offline_pages) {
57 for (const OfflinePageItem& offline_page : offline_pages) { 57 for (const OfflinePageItem& offline_page : offline_pages) {
58 Java_OfflinePageBridge_createOfflinePageAndAddToList( 58 Java_OfflinePageBridge_createOfflinePageAndAddToList(
59 env, j_result_obj, 59 env, j_result_obj,
60 ConvertUTF8ToJavaString(env, offline_page.url.spec()).obj(), 60 ConvertUTF8ToJavaString(env, offline_page.url.spec()).obj(),
61 offline_page.offline_id, 61 offline_page.offline_id,
62 ConvertUTF8ToJavaString(env, offline_page.client_id.name_space).obj(),
63 ConvertUTF8ToJavaString(env, offline_page.client_id.id).obj(),
62 ConvertUTF8ToJavaString(env, offline_page.GetOfflineURL().spec()).obj(), 64 ConvertUTF8ToJavaString(env, offline_page.GetOfflineURL().spec()).obj(),
63 offline_page.file_size, offline_page.creation_time.ToJavaTime(), 65 offline_page.file_size, offline_page.creation_time.ToJavaTime(),
64 offline_page.access_count, offline_page.last_access_time.ToJavaTime()); 66 offline_page.access_count, offline_page.last_access_time.ToJavaTime());
65 } 67 }
66 } 68 }
67 69
68 } // namespace 70 } // namespace
69 71
70 static jint GetFeatureMode(JNIEnv* env, const JavaParamRef<jclass>& clazz) { 72 static jint GetFeatureMode(JNIEnv* env, const JavaParamRef<jclass>& clazz) {
71 return static_cast<jint>(offline_pages::GetOfflinePageFeatureMode()); 73 return static_cast<jint>(offline_pages::GetOfflinePageFeatureMode());
(...skipping 208 matching lines...) Expand 10 before | Expand all | Expand 10 after
280 return; 282 return;
281 Java_OfflinePageBridge_offlinePageModelLoaded(env, obj.obj()); 283 Java_OfflinePageBridge_offlinePageModelLoaded(env, obj.obj());
282 } 284 }
283 285
284 ScopedJavaLocalRef<jobject> OfflinePageBridge::CreateOfflinePageItem( 286 ScopedJavaLocalRef<jobject> OfflinePageBridge::CreateOfflinePageItem(
285 JNIEnv* env, 287 JNIEnv* env,
286 const OfflinePageItem& offline_page) const { 288 const OfflinePageItem& offline_page) const {
287 return Java_OfflinePageBridge_createOfflinePageItem( 289 return Java_OfflinePageBridge_createOfflinePageItem(
288 env, ConvertUTF8ToJavaString(env, offline_page.url.spec()).obj(), 290 env, ConvertUTF8ToJavaString(env, offline_page.url.spec()).obj(),
289 offline_page.offline_id, 291 offline_page.offline_id,
292 ConvertUTF8ToJavaString(env, offline_page.client_id.name_space).obj(),
293 ConvertUTF8ToJavaString(env, offline_page.client_id.id).obj(),
290 ConvertUTF8ToJavaString(env, offline_page.GetOfflineURL().spec()).obj(), 294 ConvertUTF8ToJavaString(env, offline_page.GetOfflineURL().spec()).obj(),
291 offline_page.file_size, offline_page.creation_time.ToJavaTime(), 295 offline_page.file_size, offline_page.creation_time.ToJavaTime(),
292 offline_page.access_count, offline_page.last_access_time.ToJavaTime()); 296 offline_page.access_count, offline_page.last_access_time.ToJavaTime());
293 } 297 }
294 298
295 static jlong Init(JNIEnv* env, 299 static jlong Init(JNIEnv* env,
296 const JavaParamRef<jobject>& obj, 300 const JavaParamRef<jobject>& obj,
297 const JavaParamRef<jobject>& j_profile) { 301 const JavaParamRef<jobject>& j_profile) {
298 return reinterpret_cast<jlong>(new OfflinePageBridge( 302 return reinterpret_cast<jlong>(new OfflinePageBridge(
299 env, obj, ProfileAndroid::FromProfileAndroid(j_profile))); 303 env, obj, ProfileAndroid::FromProfileAndroid(j_profile)));
300 } 304 }
301 305
302 bool RegisterOfflinePageBridge(JNIEnv* env) { 306 bool RegisterOfflinePageBridge(JNIEnv* env) {
303 return RegisterNativesImpl(env); 307 return RegisterNativesImpl(env);
304 } 308 }
305 309
306 } // namespace android 310 } // namespace android
307 } // namespace offline_pages 311 } // namespace offline_pages
OLDNEW
« no previous file with comments | « chrome/android/javatests/src/org/chromium/chrome/browser/offlinepages/OfflinePageUtilsTest.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698