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

Side by Side Diff: chrome/browser/android/precache/precache_launcher.cc

Issue 1312153003: jni_generator: Pass object parameters as JavaParamRef. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase 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
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/precache/precache_launcher.h" 5 #include "chrome/browser/android/precache/precache_launcher.h"
6 6
7 #include <jni.h> 7 #include <jni.h>
8 8
9 #include "base/android/jni_android.h" 9 #include "base/android/jni_android.h"
10 #include "base/android/jni_weak_ref.h" 10 #include "base/android/jni_weak_ref.h"
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
73 precache_manager->CancelPrecaching(); 73 precache_manager->CancelPrecaching();
74 } 74 }
75 75
76 void PrecacheLauncher::OnPrecacheCompleted(bool try_again_soon) { 76 void PrecacheLauncher::OnPrecacheCompleted(bool try_again_soon) {
77 JNIEnv* env = AttachCurrentThread(); 77 JNIEnv* env = AttachCurrentThread();
78 Java_PrecacheLauncher_onPrecacheCompletedCallback( 78 Java_PrecacheLauncher_onPrecacheCompletedCallback(
79 env, weak_java_precache_launcher_.get(env).obj(), 79 env, weak_java_precache_launcher_.get(env).obj(),
80 try_again_soon ? JNI_TRUE : JNI_FALSE); 80 try_again_soon ? JNI_TRUE : JNI_FALSE);
81 } 81 }
82 82
83 static jlong Init(JNIEnv* env, jobject obj) { 83 static jlong Init(JNIEnv* env, const JavaParamRef<jobject>& obj) {
84 return reinterpret_cast<intptr_t>(new PrecacheLauncher(env, obj)); 84 return reinterpret_cast<intptr_t>(new PrecacheLauncher(env, obj));
85 } 85 }
86 86
87 // Must be run on the UI thread. 87 // Must be run on the UI thread.
88 static jboolean ShouldRun(JNIEnv* env, jobject obj) { 88 static jboolean ShouldRun(JNIEnv* env, const JavaParamRef<jobject>& obj) {
89 Profile* profile = GetProfile(); 89 Profile* profile = GetProfile();
90 PrecacheManager* precache_manager = GetPrecacheManager(profile); 90 PrecacheManager* precache_manager = GetPrecacheManager(profile);
91 return precache_manager && (precache_manager->IsInExperimentGroup() || 91 return precache_manager && (precache_manager->IsInExperimentGroup() ||
92 precache_manager->IsInControlGroup()); 92 precache_manager->IsInControlGroup());
93 } 93 }
94 94
95 bool RegisterPrecacheLauncher(JNIEnv* env) { 95 bool RegisterPrecacheLauncher(JNIEnv* env) {
96 return RegisterNativesImpl(env); 96 return RegisterNativesImpl(env);
97 } 97 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698