Adding support for sending/receiving proto bufs to dbus library.

dbus/dbus.gyp

 # Copyright (c) 2011 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': {
     'chromium_code': 1,
   },
   'targets': [
     {
@@ skipping 12 matching lines @@
         'exported_object.cc',
         'exported_object.h',
         'message.cc',
         'message.h',
         'object_proxy.cc',
         'object_proxy.h',
         'scoped_dbus_error.h',
       ],
     },
     {
+      # Protobuf compiler / generator test protocol buffer
+      'target_name': 'dbus_test_proto',
+      'type': 'static_library',
+      'sources': [ 'test_proto.proto' ],

[satorux1, 2012/02/01 19:14:57]

I don't see it in this patch. Did you forget to include this file?

[rharrison, 2012/02/01 21:33:21]

Yes, yes I did

+      'variables': {
+        'proto_out_dir': 'dbus',
+      },
+      'includes': [ '../build/protoc.gypi' ],
+    },
+    {
       # This target contains mocks that can be used to write unit tests
       # without issuing actual D-Bus calls.
       'target_name': 'dbus_test_support',
       'type': 'static_library',
       'dependencies': [
         '../build/linux/system.gyp:dbus',
         '../testing/gmock.gyp:gmock',

[satorux1, 2012/02/01 19:14:57]

Please add a dependency to the protobuf libe here.

'../third_party/protobuf/protobuf.gyp:protobuf_lite'

         'dbus',
       ],
       'sources': [
         'mock_bus.cc',
         'mock_bus.h',
         'mock_exported_object.cc',
         'mock_exported_object.h',
         'mock_object_proxy.cc',
         'mock_object_proxy.h',
       ],
       'include_dirs': [
         '..',
       ],
     },
     {
       'target_name': 'dbus_unittests',
       'type': 'executable',
       'dependencies': [
         '../base/base.gyp:test_support_base',
         '../build/linux/system.gyp:dbus',
         '../testing/gmock.gyp:gmock',
         '../testing/gtest.gyp:gtest',

[satorux1, 2012/02/01 19:14:57]

You probably need to add a dependency to the protobuf libe here as well.

'../third_party/protobuf/protobuf.gyp:protobuf_lite'

[rharrison, 2012/02/01 21:33:21]

Done.

         'dbus',
+        'dbus_test_proto',
         'dbus_test_support',
       ],
       'sources': [
         '../base/test/run_all_unittests.cc',
         'bus_unittest.cc',
         'end_to_end_async_unittest.cc',
         'end_to_end_sync_unittest.cc',
         'message_unittest.cc',
         'mock_unittest.cc',
         'test_service.cc',
         'test_service.h',
       ],
       'include_dirs': [
         '..',
       ],
     },
   ],
 }