[sync] Componentize sync: Part Final: Target 'sync' is now its own component

chrome/chrome_browser.gypi

Index: chrome/chrome_browser.gypi
diff --git a/chrome/chrome_browser.gypi b/chrome/chrome_browser.gypi
index a7a85e6e7eb5bdd6e4bb3fce9af78c91c3067b63..1039c5f12c863b188d11ef2698be7de79c6a1863 100644
--- a/chrome/chrome_browser.gypi
+++ b/chrome/chrome_browser.gypi
@@ -33,11 +33,13 @@
         '../google_apis/google_apis.gyp:google_apis',
         '../media/media.gyp:media',
         '../skia/skia.gyp:skia',
-        '../sync/protocol/sync_proto.gyp:sync_proto',
+        '../sql/sql.gyp:sql',

[akalin, 2013/01/07 20:14:51]

i assume you found a direct include of something in sql/ from browser? (just
checking)

[Raghu Simha, 2013/01/08 11:27:48]

That's right. For instance, there are numerous instances of #include
"sql/connection.h" in chrome/browser. Come to think of it, I should also add
sql.gyp:sql to export_dependent_settings. Done.

         # TODO(akalin): Depend only on sync_api from sync.
+        '../sync/sync.gyp:sync_core',

[akalin, 2013/01/07 20:14:51]

confused -- why not depend on just sync.gyp:sync?  Will that be in a follow-up
patch?

[Raghu Simha, 2013/01/08 11:27:48]

In this specific case, you're right that it might make sense to depend on
sync.gyp:sync and not the individual targets since we depend on all of them.
However, the reason I didn't do so is because we'd then have to propagate the
entire set of dependencies upstream via export_dependent_settings, and that
would cause a major increase in the number of inherited dependencies by targets
that depend on browser. Right now, the only dependency we need to export is
sync_notifier.

Let me know if you disagree, or if you think I've understood it wrong.

         '../sync/sync.gyp:sync_api',
         '../sync/sync.gyp:sync_internal_api',
         '../sync/sync.gyp:sync_notifier',
+        '../sync/sync.gyp:sync_proto',
         '../third_party/bzip2/bzip2.gyp:bzip2',
         '../third_party/cacheinvalidation/cacheinvalidation.gyp:cacheinvalidation',
         '../third_party/cacheinvalidation/cacheinvalidation.gyp:cacheinvalidation_proto_cpp',