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

Side by Side Diff: content/browser/android/background_sync_network_observer_android.cc

Issue 1489453003: jni: Pass method parameters as JavaParamRef in content. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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 "content/browser/android/background_sync_network_observer_android.h" 5 #include "content/browser/android/background_sync_network_observer_android.h"
6 6
7 #include "base/android/context_utils.h" 7 #include "base/android/context_utils.h"
8 #include "jni/BackgroundSyncNetworkObserver_jni.h" 8 #include "jni/BackgroundSyncNetworkObserver_jni.h"
9 9
10 namespace content { 10 namespace content {
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
44 BackgroundSyncNetworkObserverAndroid::Observer::~Observer() { 44 BackgroundSyncNetworkObserverAndroid::Observer::~Observer() {
45 JNIEnv* env = base::android::AttachCurrentThread(); 45 JNIEnv* env = base::android::AttachCurrentThread();
46 Java_BackgroundSyncNetworkObserver_removeObserver( 46 Java_BackgroundSyncNetworkObserver_removeObserver(
47 env, j_observer_.obj(), reinterpret_cast<jlong>(this)); 47 env, j_observer_.obj(), reinterpret_cast<jlong>(this));
48 DCHECK_CURRENTLY_ON(BrowserThread::UI); 48 DCHECK_CURRENTLY_ON(BrowserThread::UI);
49 j_observer_.Release(); 49 j_observer_.Release();
50 } 50 }
51 51
52 void BackgroundSyncNetworkObserverAndroid::Observer:: 52 void BackgroundSyncNetworkObserverAndroid::Observer::
53 NotifyConnectionTypeChanged(JNIEnv* env, 53 NotifyConnectionTypeChanged(JNIEnv* env,
54 jobject jcaller, 54 const JavaParamRef<jobject>& jcaller,
55 jint new_connection_type) { 55 jint new_connection_type) {
56 DCHECK_CURRENTLY_ON(BrowserThread::UI); 56 DCHECK_CURRENTLY_ON(BrowserThread::UI);
57 BrowserThread::PostTask( 57 BrowserThread::PostTask(
58 BrowserThread::IO, FROM_HERE, 58 BrowserThread::IO, FROM_HERE,
59 base::Bind(callback_, 59 base::Bind(callback_,
60 static_cast<net::NetworkChangeNotifier::ConnectionType>( 60 static_cast<net::NetworkChangeNotifier::ConnectionType>(
61 new_connection_type))); 61 new_connection_type)));
62 } 62 }
63 63
64 BackgroundSyncNetworkObserverAndroid::Observer::Observer( 64 BackgroundSyncNetworkObserverAndroid::Observer::Observer(
(...skipping 13 matching lines...) Expand all
78 78
79 observer_ = Observer::Create( 79 observer_ = Observer::Create(
80 base::Bind(&BackgroundSyncNetworkObserverAndroid::OnNetworkChanged, 80 base::Bind(&BackgroundSyncNetworkObserverAndroid::OnNetworkChanged,
81 weak_ptr_factory_.GetWeakPtr())); 81 weak_ptr_factory_.GetWeakPtr()));
82 } 82 }
83 83
84 BackgroundSyncNetworkObserverAndroid::~BackgroundSyncNetworkObserverAndroid() { 84 BackgroundSyncNetworkObserverAndroid::~BackgroundSyncNetworkObserverAndroid() {
85 DCHECK_CURRENTLY_ON(BrowserThread::IO); 85 DCHECK_CURRENTLY_ON(BrowserThread::IO);
86 } 86 }
87 } // namespace content 87 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698