Include sync.pb.h in sync_data.h.

chrome/chrome_tests.gypi

 # Copyright 2013 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.
 {
   'variables' : {
     'chromedriver_unittest_sources': [
       'test/chromedriver/capabilities_unittest.cc',
       'test/chromedriver/chrome/chrome_finder_unittest.cc',
       'test/chromedriver/chrome/console_logger_unittest.cc',
       'test/chromedriver/chrome/device_manager_unittest.cc',
@@ skipping 2252 matching lines @@
               ],
             }],
           ],
         }],
       ],  # conditions
     },  # target performance_browser_tests
     {
       'target_name': 'performance_ui_tests',
       'type': 'executable',
       'dependencies': [
+        'browser',

[Nico, 2014/03/07 19:13:00]

Since 'chrome' already depends on 'browser', I'd probably have a dependency on
sync here, with a comment about generated headers.

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

I've changed this to make the dependency more accurate and follow the pattern
Ryan described.  LMK if you think the updated version would benefit from a
comment here.

         'chrome',
         'chrome_resources.gyp:chrome_resources',
         'chrome_resources.gyp:chrome_strings',
         'debugger',
         'test/perf/perf_test.gyp:*',
         'test_support_common',
         'test_support_ui',
         '../base/base.gyp:base',
         '../skia/skia.gyp:skia',
         '../testing/gtest.gyp:gtest',
@@ skipping 948 matching lines @@
             'utility',
           ],
           'sources': [
             'tools/service_discovery_sniffer/service_discovery_sniffer.h',
             'tools/service_discovery_sniffer/service_discovery_sniffer.cc',
           ],
         }]
     }],
   ],  # 'conditions'
 }