Move sync performance tests into new target

chrome/chrome_tests.gypi

Index: chrome/chrome_tests.gypi
diff --git a/chrome/chrome_tests.gypi b/chrome/chrome_tests.gypi
index 9275d8e09d5b71d7a8750645b178b433dfe13cb4..e2bb581fd70521786f8afe5dc97fef426582740b 100644
--- a/chrome/chrome_tests.gypi
+++ b/chrome/chrome_tests.gypi
@@ -3447,8 +3447,6 @@
         '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',
@@ -3458,11 +3456,6 @@
         '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',
@@ -3553,6 +3546,128 @@
       ],
     },
     {
+      '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',