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

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

Issue 2237943002: Remove now-unnecessary .obj() in Java method calls. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@switch-context
Patch Set: Rebase *again* :( Created 4 years, 4 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "base/android/callback_android.h" 5 #include "base/android/callback_android.h"
6 #include "base/android/scoped_java_ref.h" 6 #include "base/android/scoped_java_ref.h"
7 #include "chrome/browser/android/offline_pages/background_scheduler_bridge.h" 7 #include "chrome/browser/android/offline_pages/background_scheduler_bridge.h"
8 #include "chrome/browser/android/offline_pages/offline_page_model_factory.h" 8 #include "chrome/browser/android/offline_pages/offline_page_model_factory.h"
9 #include "chrome/browser/android/offline_pages/request_coordinator_factory.h" 9 #include "chrome/browser/android/offline_pages/request_coordinator_factory.h"
10 #include "chrome/browser/profiles/profile.h" 10 #include "chrome/browser/profiles/profile.h"
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
55 device_conditions, base::Bind(&ProcessingDoneCallback, j_callback_ref)); 55 device_conditions, base::Bind(&ProcessingDoneCallback, j_callback_ref));
56 } 56 }
57 57
58 void BackgroundSchedulerBridge::Schedule( 58 void BackgroundSchedulerBridge::Schedule(
59 const TriggerConditions& trigger_conditions) { 59 const TriggerConditions& trigger_conditions) {
60 JNIEnv* env = base::android::AttachCurrentThread(); 60 JNIEnv* env = base::android::AttachCurrentThread();
61 ScopedJavaLocalRef<jobject> j_conditions = 61 ScopedJavaLocalRef<jobject> j_conditions =
62 CreateTriggerConditions(env, trigger_conditions.require_power_connected, 62 CreateTriggerConditions(env, trigger_conditions.require_power_connected,
63 trigger_conditions.minimum_battery_percentage, 63 trigger_conditions.minimum_battery_percentage,
64 trigger_conditions.require_unmetered_network); 64 trigger_conditions.require_unmetered_network);
65 Java_BackgroundSchedulerBridge_schedule(env, j_conditions.obj()); 65 Java_BackgroundSchedulerBridge_schedule(env, j_conditions);
66 } 66 }
67 67
68 void BackgroundSchedulerBridge::Unschedule() { 68 void BackgroundSchedulerBridge::Unschedule() {
69 JNIEnv* env = base::android::AttachCurrentThread(); 69 JNIEnv* env = base::android::AttachCurrentThread();
70 Java_BackgroundSchedulerBridge_unschedule(env); 70 Java_BackgroundSchedulerBridge_unschedule(env);
71 } 71 }
72 72
73 ScopedJavaLocalRef<jobject> BackgroundSchedulerBridge::CreateTriggerConditions( 73 ScopedJavaLocalRef<jobject> BackgroundSchedulerBridge::CreateTriggerConditions(
74 JNIEnv* env, 74 JNIEnv* env,
75 bool require_power_connected, 75 bool require_power_connected,
76 int minimum_battery_percentage, 76 int minimum_battery_percentage,
77 bool require_unmetered_network) const { 77 bool require_unmetered_network) const {
78 return Java_BackgroundSchedulerBridge_createTriggerConditions( 78 return Java_BackgroundSchedulerBridge_createTriggerConditions(
79 env, require_power_connected, minimum_battery_percentage, 79 env, require_power_connected, minimum_battery_percentage,
80 require_unmetered_network); 80 require_unmetered_network);
81 } 81 }
82 82
83 bool RegisterBackgroundSchedulerBridge(JNIEnv* env) { 83 bool RegisterBackgroundSchedulerBridge(JNIEnv* env) {
84 return RegisterNativesImpl(env); 84 return RegisterNativesImpl(env);
85 } 85 }
86 86
87 } // namespace android 87 } // namespace android
88 } // namespace offline_pages 88 } // namespace offline_pages
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698