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

Side by Side Diff: components/cronet/android/cronet_library_loader.cc

Issue 1839803002: Remove net & url small, iOS ICU alternatives, unit tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Resolved rebase conflicts Created 4 years, 8 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 "components/cronet/android/cronet_library_loader.h" 5 #include "components/cronet/android/cronet_library_loader.h"
6 6
7 #include <jni.h> 7 #include <jni.h>
8 #include <vector>
8 9
9 #include "base/android/base_jni_onload.h" 10 #include "base/android/base_jni_onload.h"
10 #include "base/android/base_jni_registrar.h" 11 #include "base/android/base_jni_registrar.h"
11 #include "base/android/jni_android.h" 12 #include "base/android/jni_android.h"
12 #include "base/android/jni_registrar.h" 13 #include "base/android/jni_registrar.h"
13 #include "base/android/jni_utils.h" 14 #include "base/android/jni_utils.h"
14 #include "base/android/library_loader/library_loader_hooks.h" 15 #include "base/android/library_loader/library_loader_hooks.h"
15 #include "base/command_line.h" 16 #include "base/command_line.h"
16 #include "base/logging.h" 17 #include "base/logging.h"
17 #include "base/macros.h" 18 #include "base/macros.h"
18 #include "base/message_loop/message_loop.h" 19 #include "base/message_loop/message_loop.h"
19 #include "components/cronet/android/chromium_url_request.h" 20 #include "components/cronet/android/chromium_url_request.h"
20 #include "components/cronet/android/chromium_url_request_context.h" 21 #include "components/cronet/android/chromium_url_request_context.h"
21 #include "components/cronet/android/cronet_bidirectional_stream_adapter.h" 22 #include "components/cronet/android/cronet_bidirectional_stream_adapter.h"
22 #include "components/cronet/android/cronet_upload_data_stream_adapter.h" 23 #include "components/cronet/android/cronet_upload_data_stream_adapter.h"
23 #include "components/cronet/android/cronet_url_request_adapter.h" 24 #include "components/cronet/android/cronet_url_request_adapter.h"
24 #include "components/cronet/android/cronet_url_request_context_adapter.h" 25 #include "components/cronet/android/cronet_url_request_context_adapter.h"
25 #include "components/cronet/version.h" 26 #include "components/cronet/version.h"
26 #include "jni/CronetLibraryLoader_jni.h" 27 #include "jni/CronetLibraryLoader_jni.h"
27 #include "net/android/net_jni_registrar.h" 28 #include "net/android/net_jni_registrar.h"
28 #include "net/android/network_change_notifier_factory_android.h" 29 #include "net/android/network_change_notifier_factory_android.h"
29 #include "net/base/network_change_notifier.h" 30 #include "net/base/network_change_notifier.h"
30 #include "url/url_util.h" 31 #include "url/url_util.h"
31 32
32 #if defined(USE_ICU_ALTERNATIVES_ON_ANDROID) 33 #if defined(USE_PLATFORM_ICU_ALTERNATIVES)
mef 2016/04/01 20:42:49 maybe we should enter a crbug to convert this into
33 #include "url/android/url_jni_registrar.h" // nogncheck 34 #include "url/android/url_jni_registrar.h" // nogncheck
34 #else 35 #else
35 #include "base/i18n/icu_util.h" 36 #include "base/i18n/icu_util.h"
36 #endif 37 #endif
37 38
38 namespace cronet { 39 namespace cronet {
39 namespace { 40 namespace {
40 41
41 const base::android::RegistrationMethod kCronetRegisteredMethods[] = { 42 const base::android::RegistrationMethod kCronetRegisteredMethods[] = {
42 {"BaseAndroid", base::android::RegisterJni}, 43 {"BaseAndroid", base::android::RegisterJni},
43 {"ChromiumUrlRequest", ChromiumUrlRequestRegisterJni}, 44 {"ChromiumUrlRequest", ChromiumUrlRequestRegisterJni},
44 {"ChromiumUrlRequestContext", ChromiumUrlRequestContextRegisterJni}, 45 {"ChromiumUrlRequestContext", ChromiumUrlRequestContextRegisterJni},
45 {"CronetBidirectionalStreamAdapter", 46 {"CronetBidirectionalStreamAdapter",
46 CronetBidirectionalStreamAdapter::RegisterJni}, 47 CronetBidirectionalStreamAdapter::RegisterJni},
47 {"CronetLibraryLoader", RegisterNativesImpl}, 48 {"CronetLibraryLoader", RegisterNativesImpl},
48 {"CronetUploadDataStreamAdapter", CronetUploadDataStreamAdapterRegisterJni}, 49 {"CronetUploadDataStreamAdapter", CronetUploadDataStreamAdapterRegisterJni},
49 {"CronetUrlRequestAdapter", CronetUrlRequestAdapterRegisterJni}, 50 {"CronetUrlRequestAdapter", CronetUrlRequestAdapterRegisterJni},
50 {"CronetUrlRequestContextAdapter", 51 {"CronetUrlRequestContextAdapter",
51 CronetUrlRequestContextAdapterRegisterJni}, 52 CronetUrlRequestContextAdapterRegisterJni},
52 {"NetAndroid", net::android::RegisterJni}, 53 {"NetAndroid", net::android::RegisterJni},
53 #if defined(USE_ICU_ALTERNATIVES_ON_ANDROID) 54 #if defined(USE_PLATFORM_ICU_ALTERNATIVES)
54 {"UrlAndroid", url::android::RegisterJni}, 55 {"UrlAndroid", url::android::RegisterJni},
55 #endif 56 #endif
56 }; 57 };
57 58
58 // MessageLoop on the main thread, which is where objects that receive Java 59 // MessageLoop on the main thread, which is where objects that receive Java
59 // notifications generally live. 60 // notifications generally live.
60 base::MessageLoop* g_main_message_loop = nullptr; 61 base::MessageLoop* g_main_message_loop = nullptr;
61 62
62 net::NetworkChangeNotifier* g_network_change_notifier = nullptr; 63 net::NetworkChangeNotifier* g_network_change_notifier = nullptr;
63 64
(...skipping 20 matching lines...) Expand all
84 return -1; 85 return -1;
85 } 86 }
86 return JNI_VERSION_1_6; 87 return JNI_VERSION_1_6;
87 } 88 }
88 89
89 void CronetOnUnLoad(JavaVM* jvm, void* reserved) { 90 void CronetOnUnLoad(JavaVM* jvm, void* reserved) {
90 base::android::LibraryLoaderExitHook(); 91 base::android::LibraryLoaderExitHook();
91 } 92 }
92 93
93 void CronetInitOnMainThread(JNIEnv* env, const JavaParamRef<jclass>& jcaller) { 94 void CronetInitOnMainThread(JNIEnv* env, const JavaParamRef<jclass>& jcaller) {
94 #if !defined(USE_ICU_ALTERNATIVES_ON_ANDROID) 95 #if !defined(USE_PLATFORM_ICU_ALTERNATIVES)
95 base::i18n::InitializeICU(); 96 base::i18n::InitializeICU();
96 #endif 97 #endif
97 98
98 // TODO(bengr): Remove once Data Reduction Proxy no longer needs this for 99 // TODO(bengr): Remove once Data Reduction Proxy no longer needs this for
99 // configuration information. 100 // configuration information.
100 base::CommandLine::Init(0, nullptr); 101 base::CommandLine::Init(0, nullptr);
101 DCHECK(!base::MessageLoop::current()); 102 DCHECK(!base::MessageLoop::current());
102 DCHECK(!g_main_message_loop); 103 DCHECK(!g_main_message_loop);
103 g_main_message_loop = new base::MessageLoopForUI(); 104 g_main_message_loop = new base::MessageLoopForUI();
104 base::MessageLoopForUI::current()->Start(); 105 base::MessageLoopForUI::current()->Start();
105 DCHECK(!g_network_change_notifier); 106 DCHECK(!g_network_change_notifier);
106 net::NetworkChangeNotifier::SetFactory( 107 net::NetworkChangeNotifier::SetFactory(
107 new net::NetworkChangeNotifierFactoryAndroid()); 108 new net::NetworkChangeNotifierFactoryAndroid());
108 g_network_change_notifier = net::NetworkChangeNotifier::Create(); 109 g_network_change_notifier = net::NetworkChangeNotifier::Create();
109 } 110 }
110 111
111 ScopedJavaLocalRef<jstring> GetCronetVersion( 112 ScopedJavaLocalRef<jstring> GetCronetVersion(
112 JNIEnv* env, 113 JNIEnv* env,
113 const JavaParamRef<jclass>& jcaller) { 114 const JavaParamRef<jclass>& jcaller) {
114 return base::android::ConvertUTF8ToJavaString(env, CRONET_VERSION); 115 return base::android::ConvertUTF8ToJavaString(env, CRONET_VERSION);
115 } 116 }
116 117
117 } // namespace cronet 118 } // namespace cronet
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698