Split cronet.jar into cronet.jar and cronet_api.jar, preserve full libcronet.so

components/cronet.gypi

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   'conditions': [
     ['OS=="android" and use_icu_alternatives_on_android==1', {
       # TODO(mef): Figure out what needs to be done for gn script.
       'targets': [
         {
@@ skipping 135 matching lines @@
                 'dependencies!': [
                   '../base/base.gyp:base_i18n',
                   '../third_party/icu/icu.gyp:icui18n',
                   '../third_party/icu/icu.gyp:icuuc',
                 ]
               },
             ],
           ],
         },
         {
+          '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 Chromium equivalents?  Perhaps a
rename (And some comments?) for these targets is in order?

[mef, 2014/08/12 16:36:11]

Renamed cronet_api into cronet_stub to reflect the fact that it also provides
basic implementation.

+              '**/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' ],
         },
         {
           'target_name': 'cronet_package',
           'type': 'none',
           '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',
               'jar_excluded_classes': [
                 '*/library_loader/*',
               ],
               'jar_extract_stamp': '<(intermediate_dir)/jar_extract.stamp',
               'cronet_jar_stamp': '<(intermediate_dir)/cronet_jar.stamp',
           },
           'actions': [
             {
               'action_name': 'strip libcronet',
               'inputs': ['<(SHARED_LIB_DIR)/<(native_lib)'],
               'outputs': ['<(package_dir)/libs/<(android_app_abi)/<(native_lib)'],
               'action': [
                 '<(android_strip)',
                 '--strip-unneeded',
                 '<@(_inputs)',
                 '-o',
                 '<@(_outputs)',
               ],
             },
             {
               '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': [
                 'python',
                 'cronet/tools/extract_from_jars.py',
                 '--classes-dir=<(jar_extract_dir)',
                 '--jars=<@(_inputs)',
                 '--stamp=<(jar_extract_stamp)',
               ],
             },
@@ skipping 52 matching lines @@
               ],
             },
           ],
           'copies': [
             {
               'destination': '<(package_dir)',
               'files': [
                 '../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)',
               ],
             },
           ],
         },
         {
           'target_name': 'cronet_sample_apk',
           'type': 'none',
           'dependencies': [
             'cronet',
           ],
@@ skipping 35 matching lines @@
             'java_in_dir': 'cronet/android/sample/javatests',
             'resource_dir': 'cronet/android/sample/res',
             'is_test_apk': 1,
           },
           'includes': [ '../build/java_apk.gypi' ],
         },
       ],
     }],  # OS=="android"
   ],
 }