[Sync] Move 'syncapi_core' and 'sync_unit_tests' targets to sync/

chrome/chrome_browser.gypi

 # Copyright (c) 2012 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.
 
 {
   'targets': [
     {
       'target_name': 'browser',
       'type': 'static_library',
       'variables': { 'enable_wexit_time_destructors': 1, },
       'dependencies': [
         'app/policy/cloud_policy_codegen.gyp:policy',
         'autofill_regexes',
         'browser_extensions',
         '../sync/protocol/sync_proto.gyp:sync_proto',
         'cert_logger_proto',
         'chrome_resources.gyp:chrome_extra_resources',
         'chrome_resources.gyp:chrome_resources',
         'chrome_resources.gyp:chrome_strings',
         'chrome_resources.gyp:platform_locale_settings',
         'chrome_resources.gyp:theme_resources',
         'common',
         'common/extensions/api/api.gyp:api',
         'common_net',
         'debugger',
         'in_memory_url_index_cache_proto',
         'installer_util',
         'safe_browsing_proto',
         'safe_browsing_report_proto',
-        # TODO(sync): Make browser not depend on syncapi_core directly.
-        'syncapi_core',
         'syncapi_service',
         'feedback_proto',
         'gdata_proto',
         '../build/temp_gyp/googleurl.gyp:googleurl',
         '../content/content.gyp:content_browser',
         '../crypto/crypto.gyp:crypto',
         '../media/media.gyp:media',
         '../net/net.gyp:net',
         '../ppapi/ppapi_internal.gyp:ppapi_proxy',  # For PpapiMsg_LoadPlugin
         '../printing/printing.gyp:printing',
         '../skia/skia.gyp:skia',
+        # TODO(sync): Make browser not depend on syncapi_core directly.

[tim (not reviewing), 2012/04/20 21:09:12]

Hrm, what was our thinking here again? Browser/sync/glue needs to depend on
internal_api...

[akalin, 2012/04/20 21:55:57]

I thought someone else added this TODO, but it turns out to have been me.  I
have no idea what I was thinking, so removed.

+        '../sync/sync.gyp:syncapi_core',
         '../third_party/bzip2/bzip2.gyp:bzip2',
         '../third_party/cld/cld.gyp:cld',
         '../third_party/expat/expat.gyp:expat',
         '../third_party/hunspell/hunspell.gyp:hunspell',
         '../third_party/icu/icu.gyp:icui18n',
         '../third_party/icu/icu.gyp:icuuc',
         '../third_party/leveldatabase/leveldatabase.gyp:leveldatabase',
         '../third_party/libjingle/libjingle.gyp:libjingle',
         '../third_party/libphonenumber/libphonenumber.gyp:libphonenumber',
         '../third_party/libxml/libxml.gyp:libxml',
@@ skipping 5152 matching lines @@
       'type': 'static_library',
       'sources': [ 'browser/chromeos/gdata/gdata.proto' ],
       'variables': {
         'proto_in_dir': 'browser/chromeos/gdata',
         'proto_out_dir': 'chrome/browser/chromeos/gdata',
       },
       'includes': [ '../build/protoc.gypi' ]
     },
   ],
 }