diff options
| author | keybuk@chromium.org <keybuk@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-14 20:08:06 +0000 |
|---|---|---|
| committer | keybuk@chromium.org <keybuk@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-14 20:08:06 +0000 |
| commit | bc1b18ecfc890dde4aae0a5a5db27dde2796b25e (patch) | |
| tree | 8e37e62d98047d218bf8fdeff2f10361fbaab961 /chrome/browser/chromeos/dbus/sensors_client.cc | |
| parent | ce1402accb9a01358fc1235f1319d934cd611248 (diff) | |
| download | chromium_src-bc1b18ecfc890dde4aae0a5a5db27dde2796b25e.zip chromium_src-bc1b18ecfc890dde4aae0a5a5db27dde2796b25e.tar.gz chromium_src-bc1b18ecfc890dde4aae0a5a5db27dde2796b25e.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
Review URL: http://codereview.chromium.org/9378039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@121920 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/chromeos/dbus/sensors_client.cc')
| -rw-r--r-- | chrome/browser/chromeos/dbus/sensors_client.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/chrome/browser/chromeos/dbus/sensors_client.cc b/chrome/browser/chromeos/dbus/sensors_client.cc index ff0dd8d..f8c03f4 100644 --- a/chrome/browser/chromeos/dbus/sensors_client.cc +++ b/chrome/browser/chromeos/dbus/sensors_client.cc @@ -11,6 +11,7 @@ #include "content/public/browser/sensors_provider.h" #include "dbus/bus.h" #include "dbus/message.h" +#include "dbus/object_path.h" #include "dbus/object_proxy.h" using content::BrowserThread; @@ -31,8 +32,9 @@ class SensorsClientImpl : public SensorsClient { explicit SensorsClientImpl(dbus::Bus* bus) : sensors_proxy_(NULL), weak_ptr_factory_(this) { - sensors_proxy_ = bus->GetObjectProxy(chromeos::kSensorsServiceName, - chromeos::kSensorsServicePath); + sensors_proxy_ = bus->GetObjectProxy( + chromeos::kSensorsServiceName, + dbus::ObjectPath(chromeos::kSensorsServicePath)); sensors_proxy_->ConnectToSignal( chromeos::kSensorsServiceInterface, chromeos::kScreenOrientationChanged, |