dbus: allow unregistering of exported objects

dbus/bus.h

Index: dbus/bus.h
diff --git a/dbus/bus.h b/dbus/bus.h
index 2684dcd0235f1982ae90e7ee90f72668c5dc0eba..e16e5f885105d3909bb2706c1ca7bb4d5499b9c2 100644
--- a/dbus/bus.h
+++ b/dbus/bus.h
@@ -226,6 +226,15 @@ class Bus : public base::RefCountedThreadSafe<Bus> {
   // Must be called in the origin thread.
   virtual ExportedObject* GetExportedObject(const ObjectPath& object_path);
 
+  // Unregisters the exported object for the given object path |object_path|.
+  //
+  // Getting an exported object for the same object path after this call
+  // will return a new object, method calls on any remaining copies of the
+  // previous object will not be called.
+  //
+  // Must be called in the origin thread.
+  virtual void UnregisterExportedObject(const ObjectPath& object_path);

[satorux1, 2012/03/13 06:07:06]

This doesn't take a callback, so clients don't know when the object is
unregistered. is it ok?

[keybuk, 2012/03/13 16:01:05]

The most likely place for this to be called is from a service provider
destructor, so a callback doesn't make sense. Though you make a point that this
introduces potentials for races, I'll adjust it a little so it doesn't have that

+
   // Shuts down the bus and blocks until it's done. More specifically, this
   // function does the following:
   //
@@ -420,6 +429,9 @@ class Bus : public base::RefCountedThreadSafe<Bus> {
  private:
   friend class base::RefCountedThreadSafe<Bus>;
 
+  // Helper function used for UnregisterExportedObject().
+  void UnregisterExportedObjectInternal(const ObjectPath& object_path);
+
   // Helper function used for ShutdownOnDBusThreadAndBlock().
   void ShutdownOnDBusThreadAndBlockInternal();