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

Unified Diff: components/cronet.gypi

Issue 463493005: Split cronet.jar into cronet.jar and cronet_api.jar, preserve full libcronet.so (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | components/cronet/android/proguard.cfg » ('j') | components/cronet/android/proguard.cfg » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/cronet.gypi
diff --git a/components/cronet.gypi b/components/cronet.gypi
index c0dfb33414eb7cbe0eb8caf8b47a22b86f5c7629..b4cdf3d7f6a89229ced0de6c1472a2fffef39a03 100644
--- a/components/cronet.gypi
+++ b/components/cronet.gypi
@@ -153,16 +153,50 @@
],
},
{
+ 'target_name': 'cronet_api',
+ 'type': 'none',
+ 'dependencies': [
+ 'cronet_url_request_context_config_list',
+ 'cronet_version',
+ ],
+ 'variables': {
+ 'java_in_dir': 'cronet/android/java',
+ 'javac_includes': [
+ '**/ChunkedWritableByteChannel.java',
+ '**/HttpUrlConnection*.java',
mmenke 2014/08/12 14:48:15 Why is this part of "cronet_api", but not the Chro
mef 2014/08/12 16:36:11 Renamed cronet_api into cronet_stub to reflect the
+ '**/HttpUrlRequest*.java',
+ '**/ResponseTooLargeException.java',
+ '**/UserAgent.java',
+ # TODO(mef): Consider moving this into HttpUrlRequestConfig.
+ '**/UrlRequestContextConfig.java',
+ '**/Version.java',
+ ],
+ },
+ 'includes': [ '../build/java.gypi' ],
+ },
+ {
'target_name': 'cronet',
'type': 'none',
'dependencies': [
'../base/base.gyp:base',
- 'libcronet',
+ 'cronet_api',
'cronet_url_request_error_list',
'cronet_url_request_priority_list',
+ 'libcronet',
],
'variables': {
'java_in_dir': 'cronet/android/java',
+ 'javac_includes': [
+ '**/ChromiumUrlRequest.java',
+ '**/ChromiumUrlRequestFactory.java',
+ '**/LibraryLoader.java',
+ # TODO(mef): Merge UrlRequest*.java into ChromiumUrlRequest*.java
+ '**/UrlRequest.java',
+ '**/UrlRequestContext.java',
+ '**/UrlRequestError.java',
+ '**/UrlRequestPriority.java',
+ '**/UsedByReflection.java',
+ ],
},
'includes': [ '../build/java.gypi' ],
},
@@ -172,12 +206,15 @@
'dependencies': [
'libcronet',
'cronet',
+ 'cronet_api',
],
'variables': {
'native_lib': 'libcronet.>(android_product_extension)',
'java_lib': 'cronet.jar',
+ 'java_api_lib': 'cronet_api.jar',
'java_src_lib': 'cronet-src.jar',
'java_sample_src_lib': 'cronet-sample-src.jar',
+ 'lib_java_dir': '<(PRODUCT_DIR)/lib.java',
'package_dir': '<(PRODUCT_DIR)/cronet',
'intermediate_dir': '<(SHARED_INTERMEDIATE_DIR)/cronet',
'jar_extract_dir': '<(intermediate_dir)/cronet_jar_extract',
@@ -203,10 +240,10 @@
{
'action_name': 'extracting from jars',
'inputs': [
- '<(PRODUCT_DIR)/lib.java/<(java_lib)',
- '<(PRODUCT_DIR)/lib.java/base_java.jar',
- '<(PRODUCT_DIR)/lib.java/net_java.jar',
- '<(PRODUCT_DIR)/lib.java/url_java.jar',
+ '<(lib_java_dir)/<(java_lib)',
+ '<(lib_java_dir)/base_java.jar',
+ '<(lib_java_dir)/net_java.jar',
+ '<(lib_java_dir)/url_java.jar',
],
'outputs': ['<(jar_extract_stamp)', '<(jar_extract_dir)'],
'action': [
@@ -279,6 +316,13 @@
'../AUTHORS',
'../chrome/VERSION',
'cronet/android/proguard.cfg',
+ '<(lib_java_dir)/<(java_api_lib)'
+ ],
+ },
+ {
+ 'destination': '<(package_dir)/symbols/<(android_app_abi)',
+ 'files': [
+ '<(SHARED_LIB_DIR)/<(native_lib)',
],
},
],
« no previous file with comments | « no previous file | components/cronet/android/proguard.cfg » ('j') | components/cronet/android/proguard.cfg » ('J')

Powered by Google App Engine
This is Rietveld 408576698