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

chromeos/dbus/fake_bluetooth_media_endpoint_service_provider.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROMEOS_DBUS_FAKE_BLUETOOTH_MEDIA_ENDPOINT_SERVICE_PROVIDER_H_
 #define CHROMEOS_DBUS_FAKE_BLUETOOTH_MEDIA_ENDPOINT_SERVICE_PROVIDER_H_
 
 #include <vector>
 
 #include "base/logging.h"
 #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:
   FakeBluetoothMediaEndpointServiceProvider(const dbus::ObjectPath object_path,
                                             Delegate* delegate);
   ~FakeBluetoothMediaEndpointServiceProvider() override;
 
   // 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
   // |delegate_| will generate the response for those methods which have
   // non-void return.
   Delegate* delegate_;
 
   DISALLOW_COPY_AND_ASSIGN(FakeBluetoothMediaEndpointServiceProvider);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DBUS_FAKE_BLUETOOTH_MEDIA_ENDPOINT_SERVICE_PROVIDER_H_