Index: chrome/browser/chromeos/dbus/bluetooth_adapter_client.cc |
=================================================================== |
--- chrome/browser/chromeos/dbus/bluetooth_adapter_client.cc (revision 121922) |
+++ chrome/browser/chromeos/dbus/bluetooth_adapter_client.cc (working copy) |
@@ -13,7 +13,6 @@ |
#include "chrome/browser/chromeos/system/runtime_environment.h" |
#include "dbus/bus.h" |
#include "dbus/message.h" |
-#include "dbus/object_path.h" |
#include "dbus/object_proxy.h" |
#include "third_party/cros_system_api/dbus/service_constants.h" |
@@ -143,11 +142,11 @@ |
break; |
} |
case dbus::Message::OBJECT_PATH: { |
- dbus::ObjectPath value; |
+ std::string value; |
if (!variant_reader.PopObjectPath(&value)) { |
return false; |
} |
- dictionary->SetString(key, value.value()); |
+ dictionary->SetString(key, value); |
break; |
} |
case dbus::Message::ARRAY: { |
@@ -209,8 +208,8 @@ |
} |
// BluetoothAdapterClient override. |
- virtual void StartDiscovery(const dbus::ObjectPath& object_path) { |
- VLOG(1) << "StartDiscovery: " << object_path.value(); |
+ virtual void StartDiscovery(const std::string& object_path) { |
+ VLOG(1) << "StartDiscovery: " << object_path; |
dbus::MethodCall method_call( |
bluetooth_adapter::kBluetoothAdapterInterface, |
@@ -226,8 +225,8 @@ |
} |
// BluetoothAdapterClient override. |
- virtual void StopDiscovery(const dbus::ObjectPath& object_path) { |
- VLOG(1) << "StopDiscovery: " << object_path.value(); |
+ virtual void StopDiscovery(const std::string& object_path) { |
+ VLOG(1) << "StopDiscovery: " << object_path; |
dbus::MethodCall method_call( |
bluetooth_adapter::kBluetoothAdapterInterface, |
@@ -244,22 +243,21 @@ |
private: |
// BluetoothManagerClient::Observer override. |
- virtual void AdapterAdded(const dbus::ObjectPath& object_path) OVERRIDE { |
- VLOG(1) << "AdapterAdded: " << object_path.value(); |
+ virtual void AdapterAdded(const std::string& object_path) OVERRIDE { |
+ VLOG(1) << "AdapterAdded: " << object_path; |
} |
// BluetoothManagerClient::Observer override. |
- virtual void AdapterRemoved(const dbus::ObjectPath& object_path) OVERRIDE { |
- VLOG(1) << "AdapterRemoved: " << object_path.value(); |
+ virtual void AdapterRemoved(const std::string& object_path) OVERRIDE { |
+ VLOG(1) << "AdapterRemoved: " << object_path; |
RemoveObjectProxyForPath(object_path); |
} |
// Ensures that we have a dbus object proxy for an adapter with dbus |
// object path |object_path|, and if not, creates it and stores it in |
// our |proxy_map_| map. |
- dbus::ObjectProxy* GetObjectProxyForPath( |
- const dbus::ObjectPath& object_path) { |
- VLOG(1) << "GetObjectProxyForPath: " << object_path.value(); |
+ dbus::ObjectProxy* GetObjectProxyForPath(const std::string& object_path) { |
+ VLOG(1) << "GetObjectProxyForPath: " << object_path; |
ProxyMap::iterator it = proxy_map_.find(object_path); |
if (it != proxy_map_.end()) |
@@ -316,129 +314,126 @@ |
// Removes the dbus object proxy for the adapter with dbus object path |
// |object_path| from our |proxy_map_| map. |
- void RemoveObjectProxyForPath(const dbus::ObjectPath& object_path) { |
- VLOG(1) << "RemoveObjectProxyForPath: " << object_path.value(); |
+ void RemoveObjectProxyForPath(const std::string& object_path) { |
+ VLOG(1) << "RemoveObjectProxyForPath: " << object_path; |
proxy_map_.erase(object_path); |
} |
// Called by dbus:: when a DeviceCreated signal is received. |
- void DeviceCreatedReceived(const dbus::ObjectPath& object_path, |
+ void DeviceCreatedReceived(const std::string& object_path, |
dbus::Signal* signal) { |
DCHECK(signal); |
dbus::MessageReader reader(signal); |
- dbus::ObjectPath device_path; |
+ std::string device_path; |
if (!reader.PopObjectPath(&device_path)) { |
- LOG(ERROR) << object_path.value() |
- << ": DeviceCreated signal has incorrect parameters: " |
- << signal->ToString(); |
+ LOG(ERROR) << object_path |
+ << ": DeviceCreated signal has incorrect parameters: " |
+ << signal->ToString(); |
return; |
} |
- VLOG(1) << object_path.value() << ": Device created: " |
- << device_path.value(); |
+ VLOG(1) << object_path << ": Device created: " << device_path; |
FOR_EACH_OBSERVER(BluetoothAdapterClient::Observer, observers_, |
DeviceCreated(object_path, device_path)); |
} |
// Called by dbus:: when the DeviceCreated signal is initially connected. |
- void DeviceCreatedConnected(const dbus::ObjectPath& object_path, |
+ void DeviceCreatedConnected(const std::string& object_path, |
const std::string& interface_name, |
const std::string& signal_name, |
bool success) { |
- LOG_IF(WARNING, !success) << object_path.value() |
- << ": Failed to connect to DeviceCreated signal."; |
+ LOG_IF(WARNING, !success) << object_path |
+ << ": Failed to connect to DeviceCreated signal."; |
} |
// Called by dbus:: when a DeviceRemoved signal is received. |
- void DeviceRemovedReceived(const dbus::ObjectPath& object_path, |
+ void DeviceRemovedReceived(const std::string& object_path, |
dbus::Signal* signal) { |
DCHECK(signal); |
dbus::MessageReader reader(signal); |
- dbus::ObjectPath device_path; |
+ std::string device_path; |
if (!reader.PopObjectPath(&device_path)) { |
- LOG(ERROR) << object_path.value() |
- << ": DeviceRemoved signal has incorrect parameters: " |
- << signal->ToString(); |
+ LOG(ERROR) << object_path |
+ << ": DeviceRemoved signal has incorrect parameters: " |
+ << signal->ToString(); |
return; |
} |
- VLOG(1) << object_path.value() << ": Device removed: " |
- << device_path.value(); |
+ VLOG(1) << object_path << ": Device removed: " << device_path; |
FOR_EACH_OBSERVER(BluetoothAdapterClient::Observer, observers_, |
DeviceRemoved(object_path, device_path)); |
} |
// Called by dbus:: when the DeviceRemoved signal is initially connected. |
- void DeviceRemovedConnected(const dbus::ObjectPath& object_path, |
+ void DeviceRemovedConnected(const std::string& object_path, |
const std::string& interface_name, |
const std::string& signal_name, |
bool success) { |
- LOG_IF(WARNING, !success) << object_path.value() |
- << ": Failed to connect to DeviceRemoved signal."; |
+ LOG_IF(WARNING, !success) << object_path |
+ << ": Failed to connect to DeviceRemoved signal."; |
} |
// Called by dbus:: when a PropertyChanged signal is received. |
- void PropertyChangedReceived(const dbus::ObjectPath& object_path, |
+ void PropertyChangedReceived(const std::string& object_path, |
dbus::Signal* signal) { |
DCHECK(signal); |
dbus::MessageReader reader(signal); |
std::string property_name; |
if (!reader.PopString(&property_name)) { |
- LOG(ERROR) << object_path.value() |
- << ": PropertyChanged signal has incorrect parameters: " |
- << signal->ToString(); |
+ LOG(ERROR) << object_path |
+ << ": PropertyChanged signal has incorrect parameters: " |
+ << signal->ToString(); |
return; |
} |
if (property_name != bluetooth_adapter::kDiscoveringProperty) { |
- VLOG(1) << object_path.value() << ": PropertyChanged: " << property_name; |
+ VLOG(1) << object_path << ": PropertyChanged: " << property_name; |
// We don't care. |
return; |
} |
bool discovering = false; |
if (!reader.PopVariantOfBool(&discovering)) { |
- LOG(ERROR) << object_path.value() |
- << ": PropertyChanged signal has incorrect parameters: " |
- << signal->ToString(); |
+ LOG(ERROR) << object_path |
+ << ": PropertyChanged signal has incorrect parameters: " |
+ << signal->ToString(); |
return; |
} |
- VLOG(1) << object_path.value() << ": PropertyChanged: Discovering = " |
- << discovering; |
+ VLOG(1) << object_path << ": PropertyChanged: Discovering = " |
+ << discovering; |
FOR_EACH_OBSERVER(BluetoothAdapterClient::Observer, observers_, |
DiscoveringPropertyChanged(object_path, discovering)); |
} |
// Called by dbus:: when the PropertyChanged signal is initially connected. |
- void PropertyChangedConnected(const dbus::ObjectPath& object_path, |
+ void PropertyChangedConnected(const std::string& object_path, |
const std::string& interface_name, |
const std::string& signal_name, |
bool success) { |
- LOG_IF(WARNING, !success) |
- << object_path.value() |
+ LOG_IF(WARNING, !success) << object_path |
<< ": Failed to connect to PropertyChanged signal."; |
} |
// Called by dbus:: when a DeviceFound signal is received. |
- void DeviceFoundReceived(const dbus::ObjectPath& object_path, |
+ void DeviceFoundReceived(const std::string& object_path, |
dbus::Signal* signal) { |
DCHECK(signal); |
dbus::MessageReader reader(signal); |
std::string address; |
if (!reader.PopString(&address)) { |
- LOG(ERROR) << object_path.value() |
- << ": DeviceFound signal has incorrect parameters: " |
- << signal->ToString(); |
+ LOG(ERROR) << object_path |
+ << ": DeviceFound signal has incorrect parameters: " |
+ << signal->ToString(); |
return; |
} |
- VLOG(1) << object_path.value() << ": Device found: " << address; |
+ VLOG(1) << object_path << ": Device found: " << address; |
DictionaryValue device_properties; |
if (!PopArrayOfDictEntries(&reader, signal, &device_properties)) { |
- LOG(ERROR) << object_path.value() |
- << ": DeviceFound signal has incorrect parameters: " |
- << signal->ToString(); |
+ LOG(ERROR) << object_path |
+ << ": DeviceFound signal has incorrect parameters: " |
+ << signal->ToString(); |
return; |
} |
@@ -447,55 +442,52 @@ |
} |
// Called by dbus:: when the DeviceFound signal is initially connected. |
- void DeviceFoundConnected(const dbus::ObjectPath& object_path, |
+ void DeviceFoundConnected(const std::string& object_path, |
const std::string& interface_name, |
const std::string& signal_name, |
bool success) { |
- LOG_IF(WARNING, !success) << object_path.value() |
- << ": Failed to connect to DeviceFound signal."; |
+ LOG_IF(WARNING, !success) << object_path |
+ << ": Failed to connect to DeviceFound signal."; |
} |
// Called by dbus:: when a DeviceDisappeared signal is received. |
- void DeviceDisappearedReceived(const dbus::ObjectPath& object_path, |
+ void DeviceDisappearedReceived(const std::string& object_path, |
dbus::Signal* signal) { |
DCHECK(signal); |
dbus::MessageReader reader(signal); |
std::string address; |
if (!reader.PopString(&address)) { |
- LOG(ERROR) << object_path.value() |
- << ": DeviceDisappeared signal has incorrect parameters: " |
- << signal->ToString(); |
+ LOG(ERROR) << object_path |
+ << ": DeviceDisappeared signal has incorrect parameters: " |
+ << signal->ToString(); |
return; |
} |
- VLOG(1) << object_path.value() << ": Device disappeared: " << address; |
+ VLOG(1) << object_path << ": Device disappeared: " << address; |
FOR_EACH_OBSERVER(BluetoothAdapterClient::Observer, observers_, |
DeviceDisappeared(object_path, address)); |
} |
// Called by dbus:: when the DeviceDisappeared signal is initially connected. |
- void DeviceDisappearedConnected(const dbus::ObjectPath& object_path, |
+ void DeviceDisappearedConnected(const std::string& object_path, |
const std::string& interface_name, |
const std::string& signal_name, |
bool success) { |
- LOG_IF(WARNING, !success) |
- << object_path.value() |
+ LOG_IF(WARNING, !success) << object_path |
<< ": Failed to connect to DeviceDisappeared signal."; |
} |
// Called when a response for StartDiscovery() is received. |
- void OnStartDiscovery(const dbus::ObjectPath& object_path, |
+ void OnStartDiscovery(const std::string& object_path, |
dbus::Response* response) { |
- VLOG(1) << "OnStartDiscovery: " << object_path.value(); |
- LOG_IF(WARNING, !response) << object_path.value() |
- << ": OnStartDiscovery: failed."; |
+ VLOG(1) << "OnStartDiscovery: " << object_path; |
+ LOG_IF(WARNING, !response) << object_path << ": OnStartDiscovery: failed."; |
} |
// Called when a response for StopDiscovery() is received. |
- void OnStopDiscovery(const dbus::ObjectPath& object_path, |
+ void OnStopDiscovery(const std::string& object_path, |
dbus::Response* response) { |
- VLOG(1) << "OnStopDiscovery: " << object_path.value(); |
- LOG_IF(WARNING, !response) << object_path.value() |
- << ": OnStopDiscovery: failed."; |
+ VLOG(1) << "OnStopDiscovery: " << object_path; |
+ LOG_IF(WARNING, !response) << object_path << ": OnStopDiscovery: failed."; |
} |
// Weak pointer factory for generating 'this' pointers that might live longer |
@@ -505,7 +497,7 @@ |
dbus::Bus* bus_; |
// We maintain a collection of dbus object proxies, one for each adapter. |
- typedef std::map<const dbus::ObjectPath, dbus::ObjectProxy*> ProxyMap; |
+ typedef std::map<const std::string, dbus::ObjectProxy*> ProxyMap; |
ProxyMap proxy_map_; |
// List of observers interested in event notifications from us. |
@@ -529,13 +521,13 @@ |
} |
// BluetoothAdapterClient override. |
- virtual void StartDiscovery(const dbus::ObjectPath& object_path) { |
- VLOG(1) << "StartDiscovery: " << object_path.value(); |
+ virtual void StartDiscovery(const std::string& object_path) { |
+ VLOG(1) << "StartDiscovery: " << object_path; |
} |
// BluetoothAdapterClient override. |
- virtual void StopDiscovery(const dbus::ObjectPath& object_path) { |
- VLOG(1) << "StopDiscovery: " << object_path.value(); |
+ virtual void StopDiscovery(const std::string& object_path) { |
+ VLOG(1) << "StopDiscovery: " << object_path; |
} |
}; |