Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6688)

Unified Diff: chrome/chrome_browser.gypi

Issue 10147003: [Sync] Move 'syncapi_core' and 'sync_unit_tests' targets to sync/ (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: sync to head Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/chrome_browser.gypi
diff --git a/chrome/chrome_browser.gypi b/chrome/chrome_browser.gypi
index 3517cc427dffb2027e1330572abf739880524802..c3b2a63493a7445f4015698c7b5aa50cb962a5c6 100644
--- a/chrome/chrome_browser.gypi
+++ b/chrome/chrome_browser.gypi
@@ -27,8 +27,6 @@
'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',
@@ -40,6 +38,8 @@
'../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/syn
akalin 2012/04/20 21:55:57 I thought someone else added this TODO, but it tur
+ '../sync/sync.gyp:syncapi_core',
'../third_party/bzip2/bzip2.gyp:bzip2',
'../third_party/cld/cld.gyp:cld',
'../third_party/expat/expat.gyp:expat',
« chrome/browser/webdata/DEPS ('K') | « chrome/chrome.gyp ('k') | chrome/chrome_tests.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698