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/test_service.h | |
| 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/test_service.h')
| -rw-r--r-- | dbus/test_service.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/dbus/test_service.h b/dbus/test_service.h index 2b49229..b5a90c7 100644 --- a/dbus/test_service.h +++ b/dbus/test_service.h @@ -76,6 +76,10 @@ class TestService : public base::Thread { // Helper function for ShutdownAndBlock(). void ShutdownAndBlockInternal(); + // Called when an ownership request is completed. + void OnOwnership(const std::string& service_name, + bool success); + // Called when a method is exported. void OnExported(const std::string& interface_name, const std::string& method_name, |