Convert ChromeSyncShell to use ChromePublic infrastructure

chrome/android/chrome_apk.gyp

 # Copyright 2015 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.
 {
   'includes': [
     '../../chrome/chrome_android_paks.gypi', # Included for the list of pak resources.
     '../../build/util/version.gypi'
    ],
   'variables': {
     'chromium_code': 1,
     'package_name': 'chrome_public_apk',
     'manifest_package': 'org.chromium.chrome',
+    'sync_shell_manifest_package': 'org.chromium.chrome.sync_shell',
     'chrome_public_apk_manifest': '<(SHARED_INTERMEDIATE_DIR)/chrome_public_apk_manifest/AndroidManifest.xml',
     'chrome_public_test_apk_manifest': '<(SHARED_INTERMEDIATE_DIR)/chrome_public_test_apk_manifest/AndroidManifest.xml',
+    'chrome_sync_shell_apk_manifest': '<(SHARED_INTERMEDIATE_DIR)/chrome_sync_shell_apk_manifest/AndroidManifest.xml',
+    'chrome_sync_shell_test_apk_manifest': '<(SHARED_INTERMEDIATE_DIR)/chrome_sync_shell_test_apk_manifest/AndroidManifest.xml',
     # This list is shared with GN.
     # Defines a list of source files should be present in the open-source
     # chrome-apk but not in the published static_library which is included in the
     # real chrome for android.
     'chrome_public_app_native_sources': [
       '../browser/android/chrome_entry_point.cc',
       '../browser/android/chrome_main_delegate_staging_android_initializer.cc',
     ],
   },
   'targets': [
@@ skipping 28 matching lines @@
       },
       'all_dependent_settings': {
         'variables': {
           'additional_input_paths': ['<(jinja_outputs_zip)'],
           'dependencies_res_zip_paths': ['<(jinja_outputs_zip)'],
         },
       },
       'includes': [ '../../build/android/jinja_template.gypi' ],
     },
     {
-      # GN: //chrome/android:chrome_public
-      'target_name': 'libchrome_public',
-      'type': 'shared_library',
+      # TODO(pvalenzuela): Implement this target with GN:
+      # http://crbug.com/475612
+      'target_name': 'chrome_sync_shell_template_resources',
+      'type': 'none',
+      'variables': {
+        'jinja_inputs_base_dir': 'java/res_template',
+        'jinja_inputs': [
+          '<(jinja_inputs_base_dir)/xml/searchable.xml',
+          '<(jinja_inputs_base_dir)/xml/syncadapter.xml',
+        ],
+        'jinja_outputs_zip': '<(PRODUCT_DIR)/res.java/<(_target_name).zip',
+        'jinja_variables': [
+          'manifest_package=<(sync_shell_manifest_package)',
+        ],
+      },
+      'all_dependent_settings': {
+        'variables': {
+          'additional_input_paths': ['<(jinja_outputs_zip)'],
+          'dependencies_res_zip_paths': ['<(jinja_outputs_zip)'],
+        },
+      },
+      'includes': [ '../../build/android/jinja_template.gypi' ],
+    },
+    {
+      # The base library used in both ChromePublic and ChromeSyncShell.
+      'target_name': 'libchrome_public_base',
+      'type': 'none',
       'dependencies': [
         '../../chrome/chrome.gyp:chrome_android_core',
       ],
       'include_dirs': [
         '../..',
       ],
-      'sources': [
-        '<@(chrome_public_app_native_sources)',
-      ],
-      'ldflags': [
-        # Some android targets still depend on --gc-sections to link.
-        # TODO: remove --gc-sections for Debug builds (crbug.com/159847).
-        '-Wl,--gc-sections',
-      ],
+      'direct_dependent_settings': {
+        'ldflags': [
+          # Some android targets still depend on --gc-sections to link.
+          # TODO: remove --gc-sections for Debug builds (crbug.com/159847).
+          '-Wl,--gc-sections',
+        ],
+      },
       'conditions': [
         # TODO(yfriedman): move this DEP to chrome_android_core to be shared
         # between internal/external.
         ['cld_version==2', {
           'dependencies': [
             '../../third_party/cld_2/cld_2.gyp:cld2_dynamic',
           ],
         }],
         # conditions for order_text_section
         # Cygprofile methods need to be linked into the instrumented build.
         ['order_profiling!=0', {
           'conditions': [
             ['OS=="android"', {
               'dependencies': [ '../../tools/cygprofile/cygprofile.gyp:cygprofile' ],
             }],
           ],
         }],  # order_profiling!=0
         ['use_allocator!="none"', {
           'dependencies': [
             '../../base/allocator/allocator.gyp:allocator',
           ],
         }],
       ],
     },
     {
+      # GN: //chrome/android:chrome_public
+      'target_name': 'libchrome_public',
+      'type': 'shared_library',
+      'sources': [
+        '<@(chrome_public_app_native_sources)',
+      ],
+      'dependencies': [
+        'libchrome_public_base',
+      ],
+    },
+    {
+      # TODO(pvalenzuela): Implement this target with GN:
+      # http://crbug.com/475612
+      'target_name': 'libchrome_sync_shell',
+      'type': 'shared_library',
+      'sources': [
+        '../browser/android/chrome_entry_point.cc',
+        '../browser/android/chrome_sync_shell_main_delegate_staging_android_initializer.cc',
+        '../browser/android/chrome_sync_shell_main_delegate_staging_android.h',
+        '../browser/android/chrome_sync_shell_main_delegate_staging_android.cc',
+      ],
+      'dependencies': [
+        'libchrome_public_base',
+        '../../sync/sync.gyp:sync',
+        '../../sync/sync.gyp:test_support_sync_fake_server_android',
+      ],
+    },
+    {
       # GN: //chrome/android:chrome_public_apk_manifest
       'target_name': 'chrome_public_manifest',
       'type': 'none',
       'variables': {
         'jinja_inputs': ['java/AndroidManifest.xml'],
         'jinja_output': '<(chrome_public_apk_manifest)',
         'jinja_variables': [
           'channel=<(android_channel)',
           'configuration_policy=<(configuration_policy)',
           'manifest_package=<(manifest_package)',
           'min_sdk_version=16',
           'target_sdk_version=22',
         ],
       },
       'includes': [ '../../build/android/jinja_template.gypi' ],
     },
     {
+      # TODO(pvalenzuela): Implement this target with GN:
+      # http://crbug.com/475612
+      'target_name': 'chrome_sync_shell_manifest',
+      'type': 'none',
+      'variables': {
+        'jinja_inputs': ['java/AndroidManifest.xml'],
+        'jinja_output': '<(chrome_sync_shell_apk_manifest)',
+        'jinja_variables': [
+          'channel=<(android_channel)',
+          'configuration_policy=<(configuration_policy)',
+          'manifest_package=<(sync_shell_manifest_package)',
+          'min_sdk_version=16',
+          'target_sdk_version=22',
+        ],
+      },
+      'includes': [ '../../build/android/jinja_template.gypi' ],
+    },
+    {
       # GN: //chrome/android:chrome_public_apk
       'target_name': 'chrome_public_apk',
       'type': 'none',
       'variables': {
         'android_manifest_path': '<(chrome_public_apk_manifest)',
         'apk_name': 'ChromePublic',
         'native_lib_target': 'libchrome_public',
         'java_in_dir': 'java',
         'conditions': [
           # Only attempt loading the library from the APK for 64 bit devices
@@ skipping 17 matching lines @@
       # chrome_public_apk creates a .jar as a side effect. Any java targets
       # that need that .jar in their classpath should depend on this target,
       'target_name': 'chrome_public_apk_java',
       'type': 'none',
       'dependencies': [
         'chrome_public_apk',
       ],
       'includes': [ '../../build/apk_fake_jar.gypi' ],
     },
     {
+      # TODO(pvalenzuela): Implement this target with GN:
+      # http://crbug.com/475612
+      'target_name': 'chrome_sync_shell_apk',
+      'type': 'none',
+      'variables': {
+        'android_manifest_path': '<(chrome_sync_shell_apk_manifest)',
+        'apk_name': 'ChromeSyncShell',
+        'native_lib_target': 'libchrome_sync_shell',
+        'java_in_dir': 'java',
+        'conditions': [
+          # Only attempt loading the library from the APK for 64 bit devices
+          # until the number of 32 bit devices which don't support this
+          # approach falls to a minimal level -  http://crbug.com/390618.
+          ['component != "shared_library" and profiling==0 and (target_arch == "arm64" or target_arch == "x86_64")', {
+            'load_library_from_zip_file': '<(chrome_apk_load_library_from_zip)',
+            'load_library_from_zip': '<(chrome_apk_load_library_from_zip)',
+          }],
+        ],
+      },
+      'dependencies': [
+        'chrome_android_paks_copy',
+        'chrome_sync_shell_template_resources',
+        '../../base/base.gyp:base_java_test_support',

[newt (away), 2015/08/11 21:25:29]

why are are all test-related deps required in ChromeSyncShell.apk? Shouldn't
they just be required in ChromeSyncShellTest.apk?

[pval...(no longer on Chromium), 2015/08/12 23:25:58]

I removed the test deps (see latest patchset), and now get this barrage Proguard
errors:

https://paste.googleplex.com/6085784813699072?raw

All of these look proto related. Obviously I can't add @VisibleForTesting in all
those spots. I tried disabling Proguard for this target and it didn't work. Is
there some known fix for this type of thing?

[pval...(no longer on Chromium), 2015/08/14 21:51:10]

I went with approach #1 from your message (add the dep to the main APK with a
comment).

+        '../chrome.gyp:chrome_java',
+        '../../sync/sync.gyp:test_support_sync_proto_java',
+        '../../sync/sync.gyp:sync',
+        '../../sync/sync.gyp:sync_java',
+        '../../sync/sync.gyp:sync_java_test_support',
+      ],
+      'includes': [ 'chrome_apk.gypi' ],
+    },
+    {
+      # GN: N/A
+      # chrome_sync_shell_apk creates a .jar as a side effect. Any java targets
+      # that need that .jar in their classpath should depend on this target.
+      'target_name': 'chrome_sync_shell_apk_java',
+      'type': 'none',
+      'dependencies': [
+        'chrome_sync_shell_apk',
+      ],
+      'includes': [ '../../build/apk_fake_jar.gypi' ],
+    },
+    {
       # GN: //chrome/android:chrome_shared_test_java
       # This target is for sharing tests between both upstream and internal
       # trees until sufficient test coverage is upstream.
       'target_name': 'chrome_shared_test_java',
       'type': 'none',
       'variables': {
         'java_in_dir': 'javatests',
       },
       'dependencies': [
         '../../base/base.gyp:base_java',
         '../../base/base.gyp:base_java_test_support',
         '../../chrome/chrome.gyp:chrome_java',
         '../../chrome/chrome.gyp:chrome_java_test_support',
         '../../components/components.gyp:invalidation_javatests',
         '../../components/components.gyp:offline_pages_enums_java',
         '../../components/components.gyp:precache_javatests',
         '../../components/components.gyp:web_contents_delegate_android_java',
         '../../content/content_shell_and_tests.gyp:content_java_test_support',
         '../../net/net.gyp:net_java',
         '../../net/net.gyp:net_java_test_support',
         '../../sync/sync.gyp:sync_java_test_support',
-        '../../sync/sync.gyp:sync_javatests',
         '../../third_party/android_tools/android_tools.gyp:android_support_v7_appcompat_javalib',
         '../../ui/android/ui_android.gyp:ui_javatests',
       ],
       'includes': [ '../../build/java.gypi' ],
     },
     {
       # GN: //chrome/android:chrome_public_test_apk_manifest
       'target_name': 'chrome_public_test_apk_manifest',
       'type': 'none',
       'variables': {
         'jinja_inputs': ['javatests/AndroidManifest.xml'],
         'jinja_output': '<(chrome_public_test_apk_manifest)',
         'jinja_variables': [
           'manifest_package=<(manifest_package)',
         ],
       },
       'includes': [ '../../build/android/jinja_template.gypi' ],
     },
     {
+      # TODO(pvalenzuela): Implement this target with GN:
+      # http://crbug.com/475612
+      'target_name': 'chrome_sync_shell_test_apk_manifest',
+      'type': 'none',
+      'variables': {
+        'jinja_inputs': ['sync_shell/javatests/AndroidManifest.xml'],
+        'jinja_output': '<(chrome_sync_shell_test_apk_manifest)',
+        'jinja_variables': [
+          'manifest_package=<(sync_shell_manifest_package)',
+        ],
+      },
+      'includes': [ '../../build/android/jinja_template.gypi' ],
+    },
+    {
       # GN: //chrome/android:chrome_public_test_apk
       'target_name': 'chrome_public_test_apk',
       'type': 'none',
       'dependencies': [
         'chrome_shared_test_java',
         'chrome_public_apk_java',
         '../../testing/android/on_device_instrumentation.gyp:broker_java',
         '../../testing/android/on_device_instrumentation.gyp:require_driver_apk',
       ],
       'variables': {
         'android_manifest_path': '<(chrome_public_test_apk_manifest)',
         'package_name': 'chrome_public_test',
         'java_in_dir': 'javatests',
         'java_in_dir_suffix': '/src_dummy',
         'apk_name': 'ChromePublicTest',
         'is_test_apk': 1,
         'test_type': 'instrumentation',
         'isolate_file': '../chrome_public_test_apk.isolate',
       },
       'includes': [
         '../../build/java_apk.gypi',
         '../../build/android/test_runner.gypi',
       ],
     },
+    {
+      # TODO(pvalenzuela): Implement this target with GN:
+      # http://crbug.com/475612
+      'target_name': 'chrome_sync_shell_test_apk',
+      'type': 'none',
+      'dependencies': [
+        'chrome_sync_shell_apk_java',
+        '../../base/base.gyp:base_java',
+        '../../base/base.gyp:base_java_test_support',
+        '../../chrome/chrome.gyp:chrome_java',
+        '../../chrome/chrome.gyp:chrome_java_test_support',
+        '../../sync/sync.gyp:sync_java',
+        '../../sync/sync.gyp:sync_java_test_support',
+        '../../sync/sync.gyp:sync_javatests',
+        '../../sync/sync.gyp:test_support_sync_proto_java',
+        '../../testing/android/on_device_instrumentation.gyp:broker_java',
+        '../../testing/android/on_device_instrumentation.gyp:require_driver_apk',
+        '../../third_party/android_tools/android_tools.gyp:android_support_v7_appcompat_javalib',

[newt (away), 2015/08/11 21:25:29]

remove this. shouldn't be needed.

[pval...(no longer on Chromium), 2015/08/14 21:51:10]

Done.

+        '../../ui/android/ui_android.gyp:ui_javatests',

[newt (away), 2015/08/11 21:25:29]

Remove this dep. Including "ui_javatests" will cause all the ui javatests to be
included in ChromeShellSyncTest, which we don't need and will waste lots of bot
time.

[pval...(no longer on Chromium), 2015/08/14 21:51:10]

Done.

+      ],
+      'variables': {
+        'android_manifest_path': '<(chrome_sync_shell_test_apk_manifest)',
+        'package_name': 'chrome_sync_shell_test',
+        'java_in_dir': 'sync_shell/javatests',
+        'apk_name': 'ChromeSyncShellTest',
+        'is_test_apk': 1,
+        'test_type': 'instrumentation',
+      },
+      'includes': [
+        '../../build/java_apk.gypi',
+        '../../build/android/test_runner.gypi',
+      ],
+    },
   ],
 }
 
 # Local Variables:
 # tab-width:2
 # indent-tabs-mode:nil
 # End:
 # vim: set expandtab tabstop=2 shiftwidth=2: