chrome: dbus: support asynchronous method replies

dbus/bus.h

Index: dbus/bus.h
diff --git a/dbus/bus.h b/dbus/bus.h
index 89fba919fb35b279d7ef63551d26866db80b17eb..39f04528100d9331c3a039a821d9b8fe3894c8b6 100644
--- a/dbus/bus.h
+++ b/dbus/bus.h
@@ -100,11 +100,15 @@ class ObjectProxy;
 //
 // Exporting a method:
 //
-//   Response* Echo(dbus::MethodCall* method_call) {
+//   void Echo(dbus::MethodCall* method_call,
+//       dbus::ExportedObject::ResponseSender response_sender) {

[satorux1, 2011/11/22 22:38:17]

not aligned vertically?

// void Echo(dbus::MethodCall* method_call,
//           dbus::ExportedObject::ResponseSender response_sender) {

[Vince Laviano, 2011/11/22 23:16:22]

Done. With the previous names, SenderResponseCallback and sender_response_cb,
vertical alignment caused the line to exceed 80 chars, hence the 4 space indent
instead.

 //     // Do something with method_call.
 //     Response* response = Response::FromMethodCall(method_call);
 //     // Build response here.
-//     return response;
+//     // Can send an immediate response here to implement a synchronous service
+//     // or store the send_response_cb and send a response later to implement

[satorux1, 2011/11/22 22:38:17]

send_response_cb  -> response_sender.

[Vince Laviano, 2011/11/22 23:16:22]

Done.

+//     // an asynchronous service.
+//     response_sender.Run(response);
 //   }
 //
 //   void OnExported(const std::string& interface_name,