device/bluetooth: Implement Unregister() of BlueotoothAudioSinkChromeOS and disconnection-related c…

chromeos/dbus/fake_bluetooth_media_endpoint_service_provider.cc

Index: chromeos/dbus/fake_bluetooth_media_endpoint_service_provider.cc
diff --git a/chromeos/dbus/fake_bluetooth_media_endpoint_service_provider.cc b/chromeos/dbus/fake_bluetooth_media_endpoint_service_provider.cc
index 31f8361b72811fcdd6da2d2479acdbff1065f669..4af9a9ffd2fabac99fdf95345da9385ce40eb852 100644
--- a/chromeos/dbus/fake_bluetooth_media_endpoint_service_provider.cc
+++ b/chromeos/dbus/fake_bluetooth_media_endpoint_service_provider.cc
@@ -48,8 +48,19 @@ void FakeBluetoothMediaEndpointServiceProvider::SelectConfiguration(
 
 void FakeBluetoothMediaEndpointServiceProvider::ClearConfiguration(
     const ObjectPath& transport_path) {
+  if (transport_path !=
+      ObjectPath(FakeBluetoothMediaTransportClient::kTransportPath))
+    return;
+
   VLOG(1) << object_path_.value() << ": ClearConfiguration on "
           << transport_path.value();
+
+  // Makes the transport object invalid for the given endpoint path.
+  FakeBluetoothMediaTransportClient* transport =
+      static_cast<FakeBluetoothMediaTransportClient*>(
+          DBusThreadManager::Get()->GetBluetoothMediaTransportClient());
+  transport->SetValid(object_path_, false);

[armansito, 2015/02/19 00:37:46]

It's a bit strange that the endpoint service provider invalidates the transport
rather than the other way around.

[Miao, 2015/02/23 21:04:24]

Right. Removed the test for this function, since ClearConfiguration will be
called in one of the following cases:
(1) Directly call SetValid on the transport object.
(2) Set the media object to be invisible. This will invalidate all transport
objects.

+
   delegate_->ClearConfiguration(transport_path);
 }