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

Unified Diff: components/cronet/android/cronet_library_loader.cc

Issue 726013002: [Cronet] Hook up library loader, system proxy and network change notifier to async api. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix compilation error. Created 5 years, 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/cronet/android/cronet_library_loader.h ('k') | components/cronet/android/cronet_loader.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/cronet/android/cronet_library_loader.cc
diff --git a/components/cronet/android/cronet_loader.cc b/components/cronet/android/cronet_library_loader.cc
similarity index 65%
rename from components/cronet/android/cronet_loader.cc
rename to components/cronet/android/cronet_library_loader.cc
index ac68005a7dc53f4c0fe387e689c35a6b5f473ad0..af0ac41ba222279bcdde67c9747432a962f23026 100644
--- a/components/cronet/android/cronet_loader.cc
+++ b/components/cronet/android/cronet_library_loader.cc
@@ -2,16 +2,25 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
+#include "components/cronet/android/cronet_library_loader.h"
+
+#include <jni.h>
+
#include "base/android/base_jni_registrar.h"
#include "base/android/jni_android.h"
#include "base/android/jni_registrar.h"
#include "base/at_exit.h"
+#include "base/logging.h"
+#include "base/message_loop/message_loop.h"
#include "components/cronet/android/chromium_url_request.h"
#include "components/cronet/android/chromium_url_request_context.h"
#include "components/cronet/android/cronet_histogram_manager.h"
#include "components/cronet/android/cronet_url_request.h"
#include "components/cronet/android/cronet_url_request_context.h"
+#include "jni/CronetLibraryLoader_jni.h"
#include "net/android/net_jni_registrar.h"
+#include "net/android/network_change_notifier_factory_android.h"
+#include "net/base/network_change_notifier.h"
#include "url/android/url_jni_registrar.h"
#include "url/url_util.h"
@@ -27,6 +36,7 @@ const base::android::RegistrationMethod kCronetRegisteredMethods[] = {
{"ChromiumUrlRequest", ChromiumUrlRequestRegisterJni},
{"ChromiumUrlRequestContext", ChromiumUrlRequestContextRegisterJni},
{"CronetHistogramManager", CronetHistogramManagerRegisterJni},
+ {"CronetLibraryLoader", RegisterNativesImpl},
{"CronetUrlRequest", CronetUrlRequestRegisterJni},
{"CronetUrlRequestContext", CronetUrlRequestContextRegisterJni},
{"NetAndroid", net::android::RegisterJni},
@@ -34,6 +44,11 @@ const base::android::RegistrationMethod kCronetRegisteredMethods[] = {
};
base::AtExitManager* g_at_exit_manager = NULL;
+// MessageLoop on the main thread, which is where objects that receive Java
+// notifications generally live.
+base::MessageLoop* g_main_message_loop = nullptr;
+
+net::NetworkChangeNotifier* g_network_change_notifier = nullptr;
} // namespace
@@ -53,10 +68,6 @@ jint CronetOnLoad(JavaVM* vm, void* reserved) {
g_at_exit_manager = new base::AtExitManager();
-#if !defined(USE_ICU_ALTERNATIVES_ON_ANDROID)
- base::i18n::InitializeICU();
-#endif
-
url::Initialize();
return JNI_VERSION_1_6;
@@ -69,4 +80,23 @@ void CronetOnUnLoad(JavaVM* jvm, void* reserved) {
}
}
+void CronetInitOnMainThread(JNIEnv* env, jclass jcaller, jobject jcontext) {
+ // Set application context.
+ base::android::ScopedJavaLocalRef<jobject> scoped_context(env, jcontext);
+ base::android::InitApplicationContext(env, scoped_context);
+
+#if !defined(USE_ICU_ALTERNATIVES_ON_ANDROID)
+ base::i18n::InitializeICU();
+#endif
+
+ DCHECK(!base::MessageLoop::current());
+ DCHECK(!g_main_message_loop);
+ g_main_message_loop = new base::MessageLoopForUI();
+ base::MessageLoopForUI::current()->Start();
+ DCHECK(!g_network_change_notifier);
+ net::NetworkChangeNotifier::SetFactory(
+ new net::NetworkChangeNotifierFactoryAndroid());
+ g_network_change_notifier = net::NetworkChangeNotifier::Create();
+}
+
} // namespace cronet
« no previous file with comments | « components/cronet/android/cronet_library_loader.h ('k') | components/cronet/android/cronet_loader.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698