Chromium Code Reviews| Index: components/cronet.gypi |
| diff --git a/components/cronet.gypi b/components/cronet.gypi |
| index c0dfb33414eb7cbe0eb8caf8b47a22b86f5c7629..bbf49645047b2a1e7cb87cc76885a6dd0d007efd 100644 |
| --- a/components/cronet.gypi |
| +++ b/components/cronet.gypi |
| @@ -152,17 +152,53 @@ |
| ], |
| ], |
| }, |
| - { |
| + { # cronet_stub.jar defines HttpUrlRequest interface and provides its |
| + # its implementation using HttpUrlConnection. |
|
mmenke
2014/08/12 16:40:14
Maybe add "(Not the Chromium network stack)"
mmenke
2014/08/12 16:40:15
nit: Don't use tabs.
mef
2014/08/12 17:30:35
Oops. I swear it is vi's fault. :) Done.
|
| + 'target_name': 'cronet_stub', |
| + 'type': 'none', |
| + 'dependencies': [ |
| + 'cronet_url_request_context_config_list', |
| + 'cronet_version', |
| + ], |
| + 'variables': { |
| + 'java_in_dir': 'cronet/android/java', |
| + 'javac_includes': [ |
| + '**/ChunkedWritableByteChannel.java', |
| + '**/HttpUrlConnection*.java', |
| + '**/HttpUrlRequest*.java', |
| + '**/ResponseTooLargeException.java', |
| + '**/UserAgent.java', |
| + # TODO(mef): Consider moving this into HttpUrlRequestConfig. |
| + '**/UrlRequestContextConfig.java', |
| + '**/Version.java', |
| + ], |
| + }, |
| + 'includes': [ '../build/java.gypi' ], |
| + }, |
| + { # cronet.jar implements HttpUrlRequest interface using Chromium stack |
| + # in native libcronet.so library. |
|
mmenke
2014/08/12 16:40:14
nit: Don't use tabs.
mef
2014/08/12 17:30:35
Done.
|
| 'target_name': 'cronet', |
| 'type': 'none', |
| 'dependencies': [ |
| '../base/base.gyp:base', |
| - 'libcronet', |
| + 'cronet_stub', |
| '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 +208,15 @@ |
| 'dependencies': [ |
| 'libcronet', |
| 'cronet', |
| + 'cronet_stub', |
| ], |
| 'variables': { |
| 'native_lib': 'libcronet.>(android_product_extension)', |
| 'java_lib': 'cronet.jar', |
| + 'java_stub_lib': 'cronet_stub.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 +242,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 +318,13 @@ |
| '../AUTHORS', |
| '../chrome/VERSION', |
| 'cronet/android/proguard.cfg', |
| + '<(lib_java_dir)/<(java_stub_lib)' |
| + ], |
| + }, |
| + { |
| + 'destination': '<(package_dir)/symbols/<(android_app_abi)', |
| + 'files': [ |
| + '<(SHARED_LIB_DIR)/<(native_lib)', |
| ], |
| }, |
| ], |