Include sync.pb.h in sync_data.h.

chrome/chrome_tests_unit.gypi

Index: chrome/chrome_tests_unit.gypi
diff --git a/chrome/chrome_tests_unit.gypi b/chrome/chrome_tests_unit.gypi
index b54530149d21223261d6e92a2ff65e607a373926..d80c0c1be7ebab31cc9bb41ab82d7e53fff48c68 100644
--- a/chrome/chrome_tests_unit.gypi
+++ b/chrome/chrome_tests_unit.gypi
@@ -12,6 +12,7 @@
         # NOTE: New dependencies should generally be added in the OS!="ios"
         # dependencies block below, rather than here.
         'browser',
+        'browser_real',

[maniscalco, 2014/03/07 23:44:39]

Needed because ui_test_utils.h includes history_service.h (which is part of
browser_real).

[Nico, 2014/03/08 00:45:57]

I think browser_real is supposed to be an implementation detail of the target
'browser' (as in, all 'browser' does is depend on 'browser_real' and export its
settings). So I'm not sure the changes in this file are necessary.

[maniscalco, 2014/03/08 01:27:00]

You're right, this dependency on browser_real is redundant (browser does the
right thing and exports browser_real's dependent settings).  Removed.

However, I believe export_dependent_settings of browser below is necessary.  Of
course, I will confirm with trybots.

         'chrome_resources.gyp:chrome_resources',
         'chrome_resources.gyp:chrome_strings',
         'chrome_resources.gyp:theme_resources',
@@ -34,6 +35,7 @@
         '../ui/message_center/message_center.gyp:message_center_test_support',
       ],
       'export_dependent_settings': [
+        'browser_real',

[maniscalco, 2014/03/07 23:44:39]

history_service.h (part of browser_real) includes sync/api/syncable_service.h so
we need to export browser_real's settings.

[maniscalco, 2014/03/08 01:27:00]

Correction: we need to export browser's settings as we've dropped the redundant
dependency on browser_real.

         '../base/base.gyp:test_support_base',
         '../content/content_shell_and_tests.gyp:test_support_content',
       ],