Supporting callback for Disconnected signal.

dbus/bus.h

 // Copyright (c) 2012 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.
 
 #ifndef DBUS_BUS_H_
 #define DBUS_BUS_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 183 matching lines @@
 
   // Creates a Bus object. The actual connection will be established when
   // Connect() is called.
   explicit Bus(const Options& options);
 
   // Called when an ownership request is complete.
   // Parameters:
   // - the requested service name.
   // - whether ownership has been obtained or not.
   typedef base::Callback<void (const std::string&, bool)> OnOwnershipCallback;
+
+  // Registers a callback function which is called when the connection with
+  // dbus-daemon is closed. This is also called when the connection is closed
+  // by ShutdownAndBlock. |closure| can be null callback.
+  void SetDisconnectedCallback(const base::Closure& closure);

[satorux1, 2013/02/14 02:30:41]

What about adding the closure parameter to CloseConnection() instead?

[Seigo Nonaka, 2013/02/14 03:33:52]

I think your suggestion doesn't help my original motivation. 
The disconnection is not only from CloseConnection, such as dbus-daemon
crashing.

One way to remove this function is adding closure parameter into dbus::Options
and setting it in ctor.
WDYT?

On 2013/02/14 02:30:41, satorux1 wrote:

+
   // TODO(satorux): Remove the service name parameter as the caller of
   // RequestOwnership() knows the service name.
 
   // Gets the object proxy for the given service name and the object path.
   // The caller must not delete the returned object.
   //
   // Returns an existing object proxy if the bus object already owns the
   // object proxy for the given service name and the object path.
   // Never returns NULL.
   //
@@ skipping 108 matching lines @@
   // The public functions below are not intended to be used in client
   // code. These are used to implement ObjectProxy and ExportedObject.
   //
 
   // Connects the bus to the dbus-daemon.
   // Returns true on success, or the bus is already connected.
   //
   // BLOCKING CALL.
   virtual bool Connect();
 
+  // Disconnects the bus from the dbus-daemon.
+  // No operation if the bus type is not a private bus or the bus is already
+  // disconnected.
+  //
+  // BLOCKING CALL.
+  virtual void CloseConnection();
+
   // Requests the ownership of the service name given by |service_name|.
   // See also RequestOwnershipAndBlock().
   //
   // |on_ownership_callback| is called when the service name is obtained
   // or failed to be obtained, in the origin thread.
   //
   // Must be called in the origin thread.
   virtual void RequestOwnership(const std::string& service_name,
                                 OnOwnershipCallback on_ownership_callback);
 
@@ skipping 252 matching lines @@
 
   bool async_operations_set_up_;
   bool shutdown_completed_;
 
   // Counters to make sure that OnAddWatch()/OnRemoveWatch() and
   // OnAddTimeout()/OnRemoveTimeou() are balanced.
   int num_pending_watches_;
   int num_pending_timeouts_;
 
   std::string address_;
+  base::Closure on_disconnected_closure_;
 
   DISALLOW_COPY_AND_ASSIGN(Bus);
 };
 
 }  // namespace dbus
 
 #endif  // DBUS_BUS_H_