diff options
| author | keybuk@google.com <keybuk@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-14 21:29:06 +0000 |
|---|---|---|
| committer | keybuk@google.com <keybuk@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-14 21:29:06 +0000 |
| commit | 216ed0bef820f496f853510c6fb274a817257ea6 (patch) | |
| tree | 04f3091bce4100749e7d167d033a69a61b9a6a90 /dbus/object_proxy.cc | |
| parent | d8f0b2d5f854cde9556fb4a9bc1a61e636aeba8e (diff) | |
| download | chromium_src-216ed0bef820f496f853510c6fb274a817257ea6.zip chromium_src-216ed0bef820f496f853510c6fb274a817257ea6.tar.gz chromium_src-216ed0bef820f496f853510c6fb274a817257ea6.tar.bz2 | |
dbus: add ObjectPath type
Rather than use std::string for object paths, add a dbus::ObjectPath type
that wraps one while allowing more type-safety. This solves all sorts of
issues with confusing object paths for strings, and allows us to do
Properties code using templates disambiguating them from strings.
BUG=chromium:109194
TEST=built and run tests
Change-Id: Icaf6f19daea4af23a9d2ec0ed76d2cbd379d680e
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=121920
Review URL: https://chromiumcodereview.appspot.com/9378039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@121941 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'dbus/object_proxy.cc')
| -rw-r--r-- | dbus/object_proxy.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/dbus/object_proxy.cc b/dbus/object_proxy.cc index 58d679c..32f6a60 100644 --- a/dbus/object_proxy.cc +++ b/dbus/object_proxy.cc @@ -13,6 +13,7 @@ #include "base/threading/thread.h" #include "base/threading/thread_restrictions.h" #include "dbus/message.h" +#include "dbus/object_path.h" #include "dbus/object_proxy.h" #include "dbus/scoped_dbus_error.h" @@ -42,7 +43,7 @@ namespace dbus { ObjectProxy::ObjectProxy(Bus* bus, const std::string& service_name, - const std::string& object_path, + const ObjectPath& object_path, int options) : bus_(bus), service_name_(service_name), |