Move sync performance tests into new target

chrome/chrome_tests.gypi

 # Copyright (c) 2011 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' : {
     'pyautolib_sources': [
       'app/chrome_command_ids.h',
       'app/chrome_dll_resource.h',
       'common/automation_constants.h',
@@ skipping 3429 matching lines @@
         'test/live_sync/live_sessions_sync_test.cc',
         'test/live_sync/live_sessions_sync_test.h',
         'test/live_sync/live_themes_sync_test.cc',
         'test/live_sync/live_themes_sync_test.h',
         'test/live_sync/live_typed_urls_sync_test.cc',
         'test/live_sync/live_typed_urls_sync_test.h',
         'test/live_sync/live_sync_extension_helper.cc',
         'test/live_sync/live_sync_extension_helper.h',
         'test/live_sync/live_sync_test.cc',
         'test/live_sync/live_sync_test.h',
-        'test/live_sync/live_sync_timing_helper.cc',
-        'test/live_sync/live_sync_timing_helper.h',
         'test/live_sync/many_client_live_bookmarks_sync_test.cc',
         'test/live_sync/many_client_live_passwords_sync_test.cc',
         'test/live_sync/many_client_live_preferences_sync_test.cc',
         'test/live_sync/migration_errors_test.cc',
         'test/live_sync/multiple_client_live_bookmarks_sync_test.cc',
         'test/live_sync/multiple_client_live_passwords_sync_test.cc',
         'test/live_sync/multiple_client_live_preferences_sync_test.cc',
         'test/live_sync/multiple_client_live_sessions_sync_test.cc',
         'test/live_sync/multiple_client_live_typed_urls_sync_test.cc',
-        'test/live_sync/performance_live_autofill_sync_test.cc',
-        'test/live_sync/performance_live_bookmarks_sync_test.cc',
-        'test/live_sync/performance_live_extensions_sync_test.cc',
-        'test/live_sync/performance_live_passwords_sync_test.cc',
-        'test/live_sync/performance_live_typed_urls_sync_test.cc',
         'test/live_sync/single_client_live_apps_sync_test.cc',
         'test/live_sync/single_client_live_bookmarks_sync_test.cc',
         'test/live_sync/single_client_live_extensions_sync_test.cc',
         'test/live_sync/single_client_live_passwords_sync_test.cc',
         'test/live_sync/single_client_live_preferences_sync_test.cc',
         'test/live_sync/single_client_live_sessions_sync_test.cc',
         'test/live_sync/single_client_live_themes_sync_test.cc',
         'test/live_sync/single_client_live_typed_urls_sync_test.cc',
         'test/live_sync/two_client_live_apps_sync_test.cc',
         'test/live_sync/two_client_live_autofill_sync_test.cc',
@@ skipping 70 matching lines @@
           ],
         }],
         ['toolkit_views==1', {
           'dependencies': [
             '../views/views.gyp:views',
           ],
         }],
       ],
     },
     {
+      'target_name': 'sync_performance_tests',
+      'type': 'executable',
+      'dependencies': [

[Raghu Simha, 2011/07/15 07:54:28]

I wonder if we can simply depend on "sync_integration_tests" instead of
explicitly listing dependencies here. I suggest adding someone who knows this
file well as a reviewer.

[braffert, 2011/07/15 21:37:41]

Simply replacing them all with sync_integration_tests does not work.  I don't
think it makes sense to include executable targets here, unless you actually
need to run that output.

+        'browser',
+        'browser/sync/protocol/sync_proto.gyp:sync_proto_cpp',
+        'chrome',
+        'chrome_resources',
+        'common',
+        'profile_import',
+        'renderer',
+        'chrome_strings',
+        'test_support_common',
+        '../net/net.gyp:net',
+        '../net/net.gyp:net_test_support',
+        '../printing/printing.gyp:printing',
+        '../skia/skia.gyp:skia',
+        '../testing/gmock.gyp:gmock',
+        '../testing/gtest.gyp:gtest',
+        '../third_party/icu/icu.gyp:icui18n',
+        '../third_party/icu/icu.gyp:icuuc',
+        '../third_party/npapi/npapi.gyp:npapi',
+        '../third_party/WebKit/Source/WebKit/chromium/WebKit.gyp:webkit',
+      ],
+      'include_dirs': [
+        '..',
+        '<(INTERMEDIATE_DIR)',
+        '<(protoc_out_dir)',
+      ],
+      'defines': [ 'HAS_OUT_OF_PROC_TEST_RUNNER' ],
+      'sources': [
+        'browser/password_manager/password_form_data.cc',
+        'test/out_of_proc_test_runner.cc',
+        'test/live_sync/bookmark_model_verifier.cc',
+        'test/live_sync/bookmark_model_verifier.h',
+        'test/live_sync/live_autofill_sync_test.cc',
+        'test/live_sync/live_autofill_sync_test.h',
+        'test/live_sync/live_bookmarks_sync_test.cc',
+        'test/live_sync/live_bookmarks_sync_test.h',
+        'test/live_sync/live_extensions_sync_test.cc',
+        'test/live_sync/live_extensions_sync_test.h',
+        'test/live_sync/live_passwords_sync_test.cc',
+        'test/live_sync/live_passwords_sync_test.h',
+        'test/live_sync/live_typed_urls_sync_test.cc',
+        'test/live_sync/live_typed_urls_sync_test.h',
+        'test/live_sync/live_sync_extension_helper.cc',
+        'test/live_sync/live_sync_extension_helper.h',
+        'test/live_sync/live_sync_test.cc',
+        'test/live_sync/live_sync_test.h',
+        'test/live_sync/perf/autofill_sync_perf_test.cc',
+        'test/live_sync/perf/bookmarks_sync_perf_test.cc',
+        'test/live_sync/perf/extensions_sync_perf_test.cc',
+        'test/live_sync/perf/live_sync_timing_helper.cc',
+        'test/live_sync/perf/live_sync_timing_helper.h',
+        'test/live_sync/perf/passwords_sync_perf_test.cc',
+        'test/live_sync/perf/typed_urls_sync_perf_test.cc',
+      ],
+      'conditions': [
+        ['toolkit_uses_gtk == 1', {
+           'dependencies': [
+             '../build/linux/system.gyp:gtk',
+             '../build/linux/system.gyp:ssl',
+           ],
+        }],
+        ['OS=="mac"', {
+          # See the comment in this section of the unit_tests target for an
+          # explanation (crbug.com/43791 - libwebcore.a is too large to mmap).
+          'dependencies+++': [
+            '../third_party/WebKit/Source/WebCore/WebCore.gyp/WebCore.gyp:webcore',
+          ],
+          # The sync_integration_tests do not run on mac without this flag.
+          # Search for comments about "xcode_settings" elsewhere in this file.
+          'xcode_settings': {'OTHER_LDFLAGS': ['-Wl,-ObjC']},
+        }],
+        ['OS=="win"', {
+          'sources': [
+            '<(SHARED_INTERMEDIATE_DIR)/app/app_resources/app_resources.rc',
+            '<(SHARED_INTERMEDIATE_DIR)/chrome/autofill_resources.rc',
+            '<(SHARED_INTERMEDIATE_DIR)/chrome/browser_resources.rc',
+            '<(SHARED_INTERMEDIATE_DIR)/chrome/common_resources.rc',
+            '<(SHARED_INTERMEDIATE_DIR)/chrome_dll_version/chrome_dll_version.rc',
+            '<(SHARED_INTERMEDIATE_DIR)/chrome/theme_resources.rc',
+            '<(SHARED_INTERMEDIATE_DIR)/chrome/theme_resources_standard.rc',
+          ],
+          'include_dirs': [
+            '<(DEPTH)/third_party/wtl/include',
+          ],
+          'dependencies': [
+            'chrome_dll_version',
+            'installer_util_strings',
+            '../sandbox/sandbox.gyp:sandbox',
+          ],
+          'conditions': [
+            ['win_use_allocator_shim==1', {
+              'dependencies': [
+                '<(allocator_target)',
+              ],
+            }],
+          ],
+          'configurations': {
+            'Debug': {
+              'msvs_settings': {
+                'VCLinkerTool': {
+                  'LinkIncremental': '<(msvs_debug_link_nonincremental)',
+                },
+              },
+            },
+          },
+        }, { # else: OS != "win"
+          'sources!': [
+            'app/chrome_dll.rc',
+            'app/chrome_dll_version.rc.version',
+            'test/data/resource.rc',
+          ],
+        }],
+        ['toolkit_views==1', {
+          'dependencies': [
+            '../views/views.gyp:views',
+          ],
+        }],
+      ],
+    },
+    {
       # Executable that contains all the tests to be run on the GPU bots.
       'target_name': 'gpu_tests',
       'type': 'executable',
       'dependencies': [
         'browser',
         'chrome',
         'chrome_resources',
         'chrome_strings',
         'renderer',
         'test_support_common',
@@ skipping 595 matching lines @@
               # Use outputs of this action as inputs for the main target build.
               # Seems as a misnomer but makes this happy on Linux (scons).
               'process_outputs_as_sources': 1,
             },
           ],  # 'actions'
         },
       ]
     }],  # 'coverage!=0'
   ],  # 'conditions'
 }