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

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

Issue 1968563003: [Offline pages] Extracting callbacks from the Offline Page Model (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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 <memory> 7 #include <memory>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/android/jni_array.h" 10 #include "base/android/jni_array.h"
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 ConvertUTF8ToJavaString(env, offline_page.client_id.name_space).obj(), 49 ConvertUTF8ToJavaString(env, offline_page.client_id.name_space).obj(),
50 ConvertUTF8ToJavaString(env, offline_page.client_id.id).obj(), 50 ConvertUTF8ToJavaString(env, offline_page.client_id.id).obj(),
51 ConvertUTF8ToJavaString(env, offline_page.GetOfflineURL().spec()).obj(), 51 ConvertUTF8ToJavaString(env, offline_page.GetOfflineURL().spec()).obj(),
52 offline_page.file_size, offline_page.creation_time.ToJavaTime(), 52 offline_page.file_size, offline_page.creation_time.ToJavaTime(),
53 offline_page.access_count, offline_page.last_access_time.ToJavaTime()); 53 offline_page.access_count, offline_page.last_access_time.ToJavaTime());
54 } 54 }
55 } 55 }
56 56
57 void CheckPagesExistOfflineCallback( 57 void CheckPagesExistOfflineCallback(
58 const ScopedJavaGlobalRef<jobject>& j_callback_obj, 58 const ScopedJavaGlobalRef<jobject>& j_callback_obj,
59 const OfflinePageModel::CheckPagesExistOfflineResult& offline_pages) { 59 const CheckPagesExistOfflineResult& offline_pages) {
60 JNIEnv* env = base::android::AttachCurrentThread(); 60 JNIEnv* env = base::android::AttachCurrentThread();
61 61
62 std::vector<std::string> offline_pages_vector; 62 std::vector<std::string> offline_pages_vector;
63 for (const GURL& page : offline_pages) 63 for (const GURL& page : offline_pages)
64 offline_pages_vector.push_back(page.spec()); 64 offline_pages_vector.push_back(page.spec());
65 65
66 ScopedJavaLocalRef<jobjectArray> j_result_array = 66 ScopedJavaLocalRef<jobjectArray> j_result_array =
67 base::android::ToJavaArrayOfStrings(env, offline_pages_vector); 67 base::android::ToJavaArrayOfStrings(env, offline_pages_vector);
68 DCHECK(j_result_array.obj()); 68 DCHECK(j_result_array.obj());
69 69
70 Java_CheckPagesExistOfflineCallbackInternal_onResult( 70 Java_CheckPagesExistOfflineCallbackInternal_onResult(
71 env, j_callback_obj.obj(), j_result_array.obj()); 71 env, j_callback_obj.obj(), j_result_array.obj());
72 } 72 }
73 73
74 void GetAllPagesCallback( 74 void GetAllPagesCallback(
75 const ScopedJavaGlobalRef<jobject>& j_result_obj, 75 const ScopedJavaGlobalRef<jobject>& j_result_obj,
76 const ScopedJavaGlobalRef<jobject>& j_callback_obj, 76 const ScopedJavaGlobalRef<jobject>& j_callback_obj,
77 const OfflinePageModel::MultipleOfflinePageItemResult& result) { 77 const MultipleOfflinePageItemResult& result) {
78 JNIEnv* env = base::android::AttachCurrentThread(); 78 JNIEnv* env = base::android::AttachCurrentThread();
79 ToJavaOfflinePageList(env, j_result_obj.obj(), result); 79 ToJavaOfflinePageList(env, j_result_obj.obj(), result);
80 80
81 Java_MultipleOfflinePageItemCallback_onResult(env, j_callback_obj.obj(), 81 Java_MultipleOfflinePageItemCallback_onResult(env, j_callback_obj.obj(),
82 j_result_obj.obj()); 82 j_result_obj.obj());
83 } 83 }
84 84
85 void HasPagesCallback(const ScopedJavaGlobalRef<jobject>& j_callback_obj, 85 void HasPagesCallback(const ScopedJavaGlobalRef<jobject>& j_callback_obj,
86 bool result) { 86 bool result) {
87 JNIEnv* env = base::android::AttachCurrentThread(); 87 JNIEnv* env = base::android::AttachCurrentThread();
88 Java_HasPagesCallback_onResult(env, j_callback_obj.obj(), result); 88 Java_HasPagesCallback_onResult(env, j_callback_obj.obj(), result);
89 } 89 }
90 90
91 void SavePageCallback(const ScopedJavaGlobalRef<jobject>& j_callback_obj, 91 void SavePageCallback(const ScopedJavaGlobalRef<jobject>& j_callback_obj,
92 const GURL& url, 92 const GURL& url,
93 OfflinePageModel::SavePageResult result, 93 SavePageResult result,
94 int64_t offline_id) { 94 int64_t offline_id) {
95 JNIEnv* env = base::android::AttachCurrentThread(); 95 JNIEnv* env = base::android::AttachCurrentThread();
96 96
97 Java_SavePageCallback_onSavePageDone( 97 Java_SavePageCallback_onSavePageDone(
98 env, j_callback_obj.obj(), static_cast<int>(result), 98 env, j_callback_obj.obj(), static_cast<int>(result),
99 ConvertUTF8ToJavaString(env, url.spec()).obj(), offline_id); 99 ConvertUTF8ToJavaString(env, url.spec()).obj(), offline_id);
100 } 100 }
101 101
102 void DeletePageCallback(const ScopedJavaGlobalRef<jobject>& j_callback_obj, 102 void DeletePageCallback(const ScopedJavaGlobalRef<jobject>& j_callback_obj,
103 OfflinePageModel::DeletePageResult result) { 103 DeletePageResult result) {
104 JNIEnv* env = base::android::AttachCurrentThread(); 104 JNIEnv* env = base::android::AttachCurrentThread();
105 105
106 Java_DeletePageCallback_onDeletePageDone( 106 Java_DeletePageCallback_onDeletePageDone(
107 env, j_callback_obj.obj(), static_cast<int>(result)); 107 env, j_callback_obj.obj(), static_cast<int>(result));
108 } 108 }
109 109
110 110
111 } // namespace 111 } // namespace
112 112
113 static jboolean IsOfflinePagesEnabled(JNIEnv* env, 113 static jboolean IsOfflinePagesEnabled(JNIEnv* env,
(...skipping 285 matching lines...) Expand 10 before | Expand all | Expand 10 after
399 ConvertUTF8ToJavaString(env, client_id.name_space).obj(), 399 ConvertUTF8ToJavaString(env, client_id.name_space).obj(),
400 ConvertUTF8ToJavaString(env, client_id.id).obj()); 400 ConvertUTF8ToJavaString(env, client_id.id).obj());
401 } 401 }
402 402
403 bool RegisterOfflinePageBridge(JNIEnv* env) { 403 bool RegisterOfflinePageBridge(JNIEnv* env) {
404 return RegisterNativesImpl(env); 404 return RegisterNativesImpl(env);
405 } 405 }
406 406
407 } // namespace android 407 } // namespace android
408 } // namespace offline_pages 408 } // namespace offline_pages
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/android/offline_pages/offline_page_tab_helper_unittest.cc » ('j') | components/offline_pages.gypi » ('J')

Powered by Google App Engine
This is Rietveld 408576698