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

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: rebase Created 4 years, 10 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
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 196 matching lines...) Expand 10 before | Expand all | Expand 10 after
268 return; 270 return;
269 Java_OfflinePageBridge_offlinePageModelLoaded(env, obj.obj()); 271 Java_OfflinePageBridge_offlinePageModelLoaded(env, obj.obj());
270 } 272 }
271 273
272 ScopedJavaLocalRef<jobject> OfflinePageBridge::CreateOfflinePageItem( 274 ScopedJavaLocalRef<jobject> OfflinePageBridge::CreateOfflinePageItem(
273 JNIEnv* env, 275 JNIEnv* env,
274 const OfflinePageItem& offline_page) const { 276 const OfflinePageItem& offline_page) const {
275 return Java_OfflinePageBridge_createOfflinePageItem( 277 return Java_OfflinePageBridge_createOfflinePageItem(
276 env, ConvertUTF8ToJavaString(env, offline_page.url.spec()).obj(), 278 env, ConvertUTF8ToJavaString(env, offline_page.url.spec()).obj(),
277 offline_page.offline_id, 279 offline_page.offline_id,
280 ConvertUTF8ToJavaString(env, offline_page.client_id.name_space).obj(),
281 ConvertUTF8ToJavaString(env, offline_page.client_id.id).obj(),
278 ConvertUTF8ToJavaString(env, offline_page.GetOfflineURL().spec()).obj(), 282 ConvertUTF8ToJavaString(env, offline_page.GetOfflineURL().spec()).obj(),
279 offline_page.file_size, offline_page.creation_time.ToJavaTime(), 283 offline_page.file_size, offline_page.creation_time.ToJavaTime(),
280 offline_page.access_count, offline_page.last_access_time.ToJavaTime()); 284 offline_page.access_count, offline_page.last_access_time.ToJavaTime());
281 } 285 }
282 286
283 static jlong Init(JNIEnv* env, 287 static jlong Init(JNIEnv* env,
284 const JavaParamRef<jobject>& obj, 288 const JavaParamRef<jobject>& obj,
285 const JavaParamRef<jobject>& j_profile) { 289 const JavaParamRef<jobject>& j_profile) {
286 return reinterpret_cast<jlong>(new OfflinePageBridge( 290 return reinterpret_cast<jlong>(new OfflinePageBridge(
287 env, obj, ProfileAndroid::FromProfileAndroid(j_profile))); 291 env, obj, ProfileAndroid::FromProfileAndroid(j_profile)));
288 } 292 }
289 293
290 bool RegisterOfflinePageBridge(JNIEnv* env) { 294 bool RegisterOfflinePageBridge(JNIEnv* env) {
291 return RegisterNativesImpl(env); 295 return RegisterNativesImpl(env);
292 } 296 }
293 297
294 } // namespace android 298 } // namespace android
295 } // namespace offline_pages 299 } // namespace offline_pages
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698