device/bluetooth:Implement BluetoothMediaEndpointServiceProvider delegate and media-related overrid…

chromeos/dbus/fake_bluetooth_media_endpoint_service_provider.h

Index: chromeos/dbus/fake_bluetooth_media_endpoint_service_provider.h
diff --git a/chromeos/dbus/fake_bluetooth_media_endpoint_service_provider.h b/chromeos/dbus/fake_bluetooth_media_endpoint_service_provider.h
index 54acd65f3b29ed523996be9521a0974ea7260abf..3f16d34afef4613b3a41a431c5835547d1c914e0 100644
--- a/chromeos/dbus/fake_bluetooth_media_endpoint_service_provider.h
+++ b/chromeos/dbus/fake_bluetooth_media_endpoint_service_provider.h
@@ -11,12 +11,12 @@
 #include "chromeos/chromeos_export.h"
 #include "chromeos/dbus/bluetooth_media_endpoint_service_provider.h"
 #include "dbus/object_path.h"
+#include "testing/gtest/include/gtest/gtest_prod.h"
 
 namespace chromeos {
 
 // FakeBluetoothMediaEndpointServiceProvider simulates the behavior of a local
 // Bluetooth Media Endpoint object.
-// TODO(mcchou): Add the logic of the behavior.
 class CHROMEOS_EXPORT FakeBluetoothMediaEndpointServiceProvider
     : public BluetoothMediaEndpointServiceProvider {
  public:
@@ -27,15 +27,24 @@ class CHROMEOS_EXPORT FakeBluetoothMediaEndpointServiceProvider
   // Each of these calls the equivalent BluetoothMediaEnpointServiceProvider::
   // Delegate method on the object passed on construction.
   void SetConfiguration(const dbus::ObjectPath& transport_path,
-                        const dbus::MessageReader& properties);
+                        const Delegate::TransportProperties& properties);
   void SelectConfiguration(
       const std::vector<uint8_t>& capabilities,
       const Delegate::SelectConfigurationCallback& callback);
   void ClearConfiguration(const dbus::ObjectPath& transport_path);
-  void Release();
+  void Released();
+
+  // Makes the endpoint object visible or invisible.
+  void SetVisible(bool visible);

[armansito, 2015/02/10 00:17:01]

I think this is a bit strange, since this is an exported object it should always
be visible. I have a faint idea of why this might be needed but I'm getting a
bit confused by the dozen SetVisible methods that all call each other.

[Miao, 2015/02/10 22:15:21]

Indeed, an endpoint is an exported object, so this method should be removed. I
was wondering if it is possible for an endpoint to be released
accidentally...but I guess testing Released() will cover this case.

+
+  // Gets the path of the media endpoint object.
+  dbus::ObjectPath object_path() const { return object_path_; }
 
  private:
-  // The fake D-Bus object path.
+  // Indicates whether the endpoint object is visible or not.
+  bool visible_;
+
+  // The path of the media endpoint object.
   dbus::ObjectPath object_path_;
 
   // All incoming method calls are passed to |delegate_|. |callback| passed to