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

chromeos/dbus/fake_bluetooth_media_transport_client.h

Index: chromeos/dbus/fake_bluetooth_media_transport_client.h
diff --git a/chromeos/dbus/fake_bluetooth_media_transport_client.h b/chromeos/dbus/fake_bluetooth_media_transport_client.h
index f88908f6111db8919c39632a6bac9022d075118a..d7bac8bb5135299f4376c18922c02ea726b07090 100644
--- a/chromeos/dbus/fake_bluetooth_media_transport_client.h
+++ b/chromeos/dbus/fake_bluetooth_media_transport_client.h
@@ -17,6 +17,8 @@
 
 namespace chromeos {
 
+class FakeBluetoothMediaEndpointServiceProvider;
+
 class CHROMEOS_EXPORT FakeBluetoothMediaTransportClient
     : public BluetoothMediaTransportClient {
  public:
@@ -62,21 +64,29 @@ class CHROMEOS_EXPORT FakeBluetoothMediaTransportClient
                const base::Closure& callback,
                const ErrorCallback& error_callback) override;
 
-  // Makes the transport valid/invalid for a given media endpoint path. The
-  // transport object is assigned to the given endpoint if valid is true, false
+  // Makes the transport valid/invalid for a given media endpoint. The transport
+  // object is assigned to the given endpoint if valid is true, false
   // otherwise.
-  void SetValid(const dbus::ObjectPath& endpoint_path, bool valid);
+  void SetValid(FakeBluetoothMediaEndpointServiceProvider* endpoint,
+                bool valid);
+
+  // Gets the transport path associated with the given endpoint path.
+  dbus::ObjectPath GetTransportPath(dbus::ObjectPath endpoint_path);

[armansito, 2015/02/23 22:07:06]

nit: Change argument type to "const dbus::ObjectPath&". Here and everywhere else
where you pass an object path as argument.

[Miao, 2015/02/24 01:12:12]

Done.

 
  private:
   // Property callback passed while a Properties structure is created.
   void OnPropertyChanged(const std::string& property_name);
 
+  // Map of endpoints with valid transport. Each pair is composed of an endpoint
+  // path and a transport path bound to that endpoint path.
+  std::map<dbus::ObjectPath, dbus::ObjectPath> endpoints_;
+
+  // The endpoint path associated with the transport object.
+  dbus::ObjectPath endpoint_path_;
+
   // The path of the media transport object.
   dbus::ObjectPath object_path_;
 
-  // Indicates whether endpoints are assigned with the transport object.
-  std::map<dbus::ObjectPath, bool> endpoints_;
-
   // The fake property set of the media transport object.
   scoped_ptr<Properties> properties_;