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

Side by Side Diff: chrome/browser/android/interests_service.cc

Issue 1334233003: Add the JNI code in order to let Java use the InterestsFetcher. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@ntp-interests-retriever
Patch Set: Created 5 years, 3 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
(Empty)
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
3 // found in the LICENSE file.
4
5 #include "chrome/browser/android/interests_service.h"
6
7 #include "base/android/jni_android.h"
8 #include "base/android/jni_array.h"
9 #include "base/android/jni_string.h"
10 #include "base/android/scoped_java_ref.h"
11 #include "base/callback.h"
12 #include "base/command_line.h"
Marc Treib 2015/09/11 14:54:25 Not needed, also the next three (and probably some
tache 2015/09/11 15:45:09 Done.
13 #include "base/compiler_specific.h"
14 #include "base/metrics/histogram.h"
15 #include "base/metrics/sparse_histogram.h"
16 #include "base/strings/string_number_conversions.h"
17 #include "base/strings/stringprintf.h"
18 #include "base/strings/utf_string_conversions.h"
19 #include "base/time/time.h"
20 #include "chrome/browser/interests/interests_fetcher.h"
21 #include "chrome/browser/profiles/profile.h"
22 #include "chrome/browser/profiles/profile_android.h"
23 #include "content/public/browser/browser_thread.h"
24 #include "content/public/browser/url_data_source.h"
25 #include "jni/InterestsService_jni.h"
26 #include "url/gurl.h"
27
28 using base::android::AttachCurrentThread;
29 using base::android::ConvertJavaStringToUTF8;
30 using base::android::ScopedJavaGlobalRef;
31 using base::android::ScopedJavaLocalRef;
32 using base::android::ToJavaArrayOfStrings;
33
34 namespace {
35
36 const char kInterestJavaClass[] =
37 "org/chromium/chrome/browser/ntp/InterestsService$Interest";
38
39 } // end namespace
40
41
42 InterestsService::InterestsService(Profile *profile):
43 profile_(profile), weak_ptr_factory_(this) {
44 }
45
46 InterestsService::~InterestsService() {
47 }
48
49 void InterestsService::Destroy(JNIEnv* env, jobject obj) {
50 delete this;
51 }
52
53
54
55 void InterestsService::GetInterests(JNIEnv* env,
56 jobject obj,
57 jstring j_token,
58 jobject j_callback_obj) {
59 scoped_ptr<ScopedJavaGlobalRef<jobject>> j_callback(
60 new ScopedJavaGlobalRef<jobject>());
61
62 j_callback->Reset(env, j_callback_obj);
63 std::string token = ConvertJavaStringToUTF8(env, j_token);
64
65 InterestsFetcher* fetcher =
66 new InterestsFetcher(profile_->GetRequestContext(),
67 token);
68
69 InterestsFetcher::InterestsCallback callback =
70 base::Bind(&InterestsService::OnObtainedInterests,
71 weak_ptr_factory_.GetWeakPtr(),
72 fetcher,
Marc Treib 2015/09/11 14:54:25 Can you use base::Owned here? Then |fetcher| shoul
tache 2015/09/11 15:45:09 Done.
73 base::Passed(&j_callback));
74
75 fetcher->Start(callback);
76 }
77
78 // static
79 bool InterestsService::Register(JNIEnv* env) {
80 return RegisterNativesImpl(env);
81 }
82
83
84 jobjectArray InterestsService::ConvertInterestsToJava(
85 const std::vector<InterestsFetcher::Interest>& interests) {
86 JNIEnv* env = AttachCurrentThread();
87 jclass interest_class = env->FindClass(kInterestJavaClass);
88
89 jobjectArray j_interests =
90 env->NewObjectArray(interests.size(), interest_class, 0);
91
92 for (uint i = 0; i != interests.size(); i++) {
Marc Treib 2015/09/11 14:54:25 nit: size_t
tache 2015/09/11 15:45:09 Done.
93 ScopedJavaLocalRef<jobject> j_interest =
94 Java_InterestsService_createInterest(env,
95 env->NewStringUTF(interests[i].name.c_str()),
96 env->NewStringUTF(interests[i].image_url.c_str()),
97 interests[i].relevance);
98
99 env->SetObjectArrayElement(j_interests, i, j_interest.obj());
100 }
101
102 return j_interests;
103 }
104
105 void InterestsService::OnObtainedInterests(
106 InterestsFetcher* fetcher,
107 scoped_ptr<ScopedJavaGlobalRef<jobject>> j_callback,
108 const std::vector<InterestsFetcher::Interest>& interests) {
109 JNIEnv* env = AttachCurrentThread();
110
111 jobjectArray j_interests =
112 InterestsService::ConvertInterestsToJava(interests);
113
114 Java_GetInterestsCallback_onInterestsCallback(
115 env, j_callback->obj(), j_interests);
116
117 base::MessageLoop::current()->DeleteSoon(FROM_HERE, fetcher);
118 }
119
120 static jlong Init(JNIEnv* env,
121 const JavaParamRef<jobject>& jobj,
122 const JavaParamRef<jobject>& jprofile) {
123 InterestsService *interests_service = new InterestsService(
124 ProfileAndroid::FromProfileAndroid(jprofile));
125 return reinterpret_cast<intptr_t>(interests_service);
126 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698