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

chromeos/dbus/fake_bluetooth_media_transport_client.cc

 // 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.
 
 #include "chromeos/dbus/fake_bluetooth_media_transport_client.h"
 
+#include <sstream>
+
 #include "base/bind.h"
+#include "base/stl_util.h"
 #include "chromeos/dbus/bluetooth_media_client.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/dbus/fake_bluetooth_media_client.h"
+#include "chromeos/dbus/fake_bluetooth_media_endpoint_service_provider.h"
 
 using dbus::ObjectPath;
 
 namespace {
 
 // TODO(mcchou): Remove this constants once it is in cros_system_api.
 const char kBluetoothMediaTransportInterface[] = "org.bluez.MediaTransport1";
 const char kNotImplemented[] = "org.bluez.NotImplemented";
 
+ObjectPath GenerateTransportPath() {
+  static unsigned int sequence_number = 0;
+  ++sequence_number;
+  std::stringstream path;
+  path << "fake/hci0/dev_00_00_00_00_00_00/fd" << sequence_number;

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

nit: if you want to truly simulate bluez's behavior then you could actually pass
the adapter path as the prefix (as in if the adapter is hci1, this should also
start with hci1). I guess the dev_00* part doesn't really matter but I'd put
something like "*/fake_audio_source/fd" as the device path instead of all 0s for
the address.

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

Referred to chromeos::FakeBluetoothAdapterClient::kAdapterPath for now.

+  return ObjectPath(path.str());
+}
+
 }  // namespace
 
 namespace chromeos {
 
 // static
-const char FakeBluetoothMediaTransportClient::kTransportPath[] =
-    "/fake/hci0/dev_00_00_00_00_00_00/fd0";
 const char FakeBluetoothMediaTransportClient::kTransportDevicePath[] =
     "/fake/hci0/dev_00_00_00_00_00_00";

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

nit: How about "*/fake_audio_source"? Also, can this be something other than
hci0?

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

Done.

 const uint8_t FakeBluetoothMediaTransportClient::kTransportCodec = 0x00;
 const std::vector<uint8_t>
     FakeBluetoothMediaTransportClient::kTransportConfiguration = {
         0x21, 0x15, 0x33, 0x2C};
 const uint16_t FakeBluetoothMediaTransportClient::kTransportDelay = 5;
 const uint16_t FakeBluetoothMediaTransportClient::kTransportVolume = 10;
 
 FakeBluetoothMediaTransportClient::Properties::Properties(
     const PropertyChangedCallback& callback)
     : BluetoothMediaTransportClient::Properties(
@@ skipping 18 matching lines @@
   VLOG(1) << "GetAll called.";
 }
 
 void FakeBluetoothMediaTransportClient::Properties::Set(
     dbus::PropertyBase* property,
     dbus::PropertySet::SetCallback callback) {
   VLOG(1) << "Set " << property->name();
   callback.Run(false);
 }
 
-FakeBluetoothMediaTransportClient::FakeBluetoothMediaTransportClient()
-    : object_path_(ObjectPath(kTransportPath)) {
+FakeBluetoothMediaTransportClient::FakeBluetoothMediaTransportClient() {
   // TODO(mcchou): Multiple endpoints are sharing one property set for now.
   // Add property sets accordingly to separate the
   // MediaTransportPropertiesChanged events for different endpoints.
 
   // Sets fake property set with default values.
   properties_.reset(new Properties(
       base::Bind(&FakeBluetoothMediaTransportClient::OnPropertyChanged,
                  base::Unretained(this))));
   properties_->device.ReplaceValue(ObjectPath(kTransportDevicePath));
   properties_->uuid.ReplaceValue(BluetoothMediaClient::kBluetoothAudioSinkUUID);
@@ skipping 44 matching lines @@
 }
 
 void FakeBluetoothMediaTransportClient::Release(
     const ObjectPath& object_path,
     const base::Closure& callback,
     const ErrorCallback& error_callback) {
   error_callback.Run(kNotImplemented, "");
 }
 
 void FakeBluetoothMediaTransportClient::SetValid(
-    const ObjectPath& endpoint_path,
+    FakeBluetoothMediaEndpointServiceProvider* endpoint,
     bool valid) {
+  FakeBluetoothMediaClient* media = static_cast<FakeBluetoothMediaClient*>(
+      DBusThreadManager::Get()->GetBluetoothMediaClient());
+  DCHECK(media);
+
+  ObjectPath endpoint_path(endpoint->object_path());
+  if (!media->IsRegistered(endpoint_path))
+    return;
+
   if (valid) {
-    endpoints_[endpoint_path] = valid;
+    ObjectPath transport_path = GenerateTransportPath();
+    VLOG(1) << "New transport, " << transport_path.value()
+            << " is created for endpoint " << endpoint_path.value();
+    endpoints_[endpoint_path] = transport_path;
     return;
   }
-  endpoints_.erase(endpoint_path);
-
-  // TODO(mcchou): Since there is only one transport path, all observers will
-  // be notified. Shades irrelevant observers.
 
   // Notifies observers about the state change of the transport.
   FOR_EACH_OBSERVER(BluetoothMediaTransportClient::Observer, observers_,
-                    MediaTransportRemoved(object_path_));
+                    MediaTransportRemoved(endpoints_[endpoint_path]));
+  endpoints_.erase(endpoint_path);
+  endpoint->ClearConfiguration(GetTransportPath(endpoint_path));
+}
+
+ObjectPath FakeBluetoothMediaTransportClient::GetTransportPath(
+    ObjectPath endpoint_path) {
+  if (ContainsKey(endpoints_, endpoint_path))
+    return endpoints_[endpoint_path];
+  return ObjectPath("");
 }
 
 void FakeBluetoothMediaTransportClient::OnPropertyChanged(
     const std::string& property_name) {
   VLOG(1) << "Property " << property_name << " changed";
 }
 
 }  // namespace chromeos