Sync: Generalize entity injection in Android tests

sync/sync.gyp

 # Copyright 2013 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.
 
 {
   'variables': {
     'chromium_code': 1,
+    # Setting these two variables allows other targets to use the
+    # sync_proto_sources variable as the list of sync protocol buffer files.
+    'sync_proto_sources_dir': 'protocol',
+    'sync_proto_sources': [
+      '<@(sync_proto_source_paths)',
+    ],
   },
 
   'includes': [
     'sync_android.gypi',
     'sync_tests.gypi',
+    'protocol/protocol.gypi',

[Nicolas Zea, 2015/04/09 20:38:16]

nit: abc order

[pval...(no longer on Chromium), 2015/04/09 21:28:05]

Done.

   ],
 
   'targets': [
     # This target will add '-lsync_core' and '-lsync_proto' to the link line of
     # targets that depend on it.  Anything that depends on sync code should
     # declare a dependency on this target.
     {
       'target_name': 'sync',
       'type': 'none',
       'dependencies': [
@@ skipping 440 matching lines @@
       # GN version: //sync/protocol
       'target_name': 'sync_proto',
       'type': '<(component)',
       'include_dirs': [
         '..',
       ],
       'defines': [
         'SYNC_PROTO_IMPLEMENTATION',
       ],
       'sources': [
-        # NOTE: If you add a file to this list, also add it to
-        # sync/protocol/BUILD.gn
-        'protocol/app_notification_specifics.proto',
-        'protocol/app_setting_specifics.proto',
-        'protocol/app_specifics.proto',
-        'protocol/app_list_specifics.proto',
-        'protocol/article_specifics.proto',
-        'protocol/attachments.proto',
-        'protocol/autofill_specifics.proto',
-        'protocol/bookmark_specifics.proto',
-        'protocol/client_commands.proto',
-        'protocol/client_debug_info.proto',
-        'protocol/device_info_specifics.proto',
-        'protocol/dictionary_specifics.proto',
-        'protocol/encryption.proto',
-        'protocol/experiment_status.proto',
-        'protocol/experiments_specifics.proto',
-        'protocol/extension_setting_specifics.proto',
-        'protocol/extension_specifics.proto',
-        'protocol/favicon_image_specifics.proto',
-        'protocol/favicon_tracking_specifics.proto',
-        'protocol/get_updates_caller_info.proto',
-        'protocol/history_delete_directive_specifics.proto',
-        'protocol/nigori_specifics.proto',
-        'protocol/managed_user_setting_specifics.proto',
-        'protocol/managed_user_shared_setting_specifics.proto',
-        'protocol/managed_user_specifics.proto',
-        'protocol/managed_user_whitelist_specifics.proto',
-        'protocol/password_specifics.proto',
-        'protocol/preference_specifics.proto',
-        'protocol/priority_preference_specifics.proto',
-        'protocol/search_engine_specifics.proto',
-        'protocol/session_specifics.proto',
-        'protocol/sync.proto',
-        'protocol/sync_enums.proto',
-        'protocol/synced_notification_app_info_specifics.proto',
-        'protocol/synced_notification_specifics.proto',
-        'protocol/test.proto',
-        'protocol/theme_specifics.proto',
-        'protocol/typed_url_specifics.proto',
-        'protocol/unique_position.proto',
-        'protocol/wifi_credential_specifics.proto',
+        # When adding a new proto source file, add its path to the list defined
+        # in sync/protocol/protocol.gypi.
+        '<@(sync_proto_sources)',
       ],
       'variables': {
         'enable_wexit_time_destructors': 1,
         'proto_in_dir': './protocol',
         'proto_out_dir': 'sync/protocol',
         'cc_generator_options': 'dllexport_decl=SYNC_PROTO_EXPORT:',
         'cc_include': 'sync/protocol/sync_proto_export.h',
       },
       'includes': [
         '../build/protoc.gypi'
@@ skipping 21 matching lines @@
       },
       'includes': [
         '../build/protoc.gypi'
       ],
       'defines': [
         'SYNC_IMPLEMENTATION'
       ],
     },
   ],
 }