diff options
| author | keybuk@chromium.org <keybuk@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-10 01:12:52 +0000 |
|---|---|---|
| committer | keybuk@chromium.org <keybuk@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-10 01:12:52 +0000 |
| commit | 15e7b16ff24684e2678fe1b4964125b42a4c4018 (patch) | |
| tree | 5f53e19b8a27e5af437e342d5464b110f4980a48 /dbus/exported_object.cc | |
| parent | 093fcf536f77186f754119854ab729e68c31f784 (diff) | |
| download | chromium_src-15e7b16ff24684e2678fe1b4964125b42a4c4018.zip chromium_src-15e7b16ff24684e2678fe1b4964125b42a4c4018.tar.gz chromium_src-15e7b16ff24684e2678fe1b4964125b42a4c4018.tar.bz2 | |
dbus: remove service name from ExportedObject
Well-known names in D-Bus are merely aliases to unique connection ids
maintained by the bus, they have no purpose in qualifying object paths
or interfaces and it's perfectly legimiate for a client to make
requests to the unique connection id (e.g. in response to a signal,
which does not reference the well-known name of the origin connection).
Remove the service_name member from dbus::ExportedObject, from its
constructor and from dbus::Bus::GetExportedObject and require code to
call dbus::Bus::RequestOwnership if a well-known name is desired. This
requires making that function callable from the origin thread with
a callback for the return value.
BUG=chromium-os:27101
TEST=dbus_unittests
Change-Id: Ib91de8b68ad9c3b432e224a2c715f0c2ca1af463
Review URL: http://codereview.chromium.org/9668018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125970 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'dbus/exported_object.cc')
| -rw-r--r-- | dbus/exported_object.cc | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/dbus/exported_object.cc b/dbus/exported_object.cc index 730c98b..7c4bada 100644 --- a/dbus/exported_object.cc +++ b/dbus/exported_object.cc @@ -35,10 +35,8 @@ std::string GetAbsoluteMethodName( } // namespace ExportedObject::ExportedObject(Bus* bus, - const std::string& service_name, const ObjectPath& object_path) : bus_(bus), - service_name_(service_name), object_path_(object_path), object_is_registered_(false) { } @@ -65,8 +63,6 @@ bool ExportedObject::ExportMethodAndBlock( return false; if (!bus_->SetUpAsyncOperations()) return false; - if (!bus_->RequestOwnership(service_name_)) - return false; if (!Register()) return false; |