bluetooth: Shutdown BluetoothAdapter before DBus on ChromeOS.

device/bluetooth/bluetooth_chromeos_unittest.cc

 // Copyright 2013 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 "base/memory/scoped_vector.h"
 #include "base/message_loop/message_loop.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/dbus/fake_bluetooth_adapter_client.h"
 #include "chromeos/dbus/fake_bluetooth_agent_manager_client.h"
@@ skipping 3294 matching lines @@
   // Calling GetConnectionInfo for a connected device should return valid
   // results.
   fake_bluetooth_device_client_->UpdateConnectionInfo(-10, 3, 4);
   BluetoothDevice::ConnectionInfo conn_info;
   device->GetConnectionInfo(base::Bind(&SaveConnectionInfo, &conn_info));
   EXPECT_EQ(-10, conn_info.rssi);
   EXPECT_EQ(3, conn_info.transmit_power);
   EXPECT_EQ(4, conn_info.max_transmit_power);
 }
 
+// Verifies OnDBusThreadManagerShutdown shuts down the adapter as expected.
+TEST_F(BluetoothChromeOSTest, OnDBusThreadManagerShutdown) {
+  // Set up and adapter, power, discoverable, start discovery.
+  GetAdapter();
+  adapter_->SetPowered(true, base::Bind(&BluetoothChromeOSTest::Callback,
+                                        base::Unretained(this)),
+                       base::Bind(&BluetoothChromeOSTest::ErrorCallback,
+                                  base::Unretained(this)));
+  adapter_->SetDiscoverable(true, base::Bind(&BluetoothChromeOSTest::Callback,
+                                             base::Unretained(this)),
+                            base::Bind(&BluetoothChromeOSTest::ErrorCallback,
+                                       base::Unretained(this)));
+  adapter_->StartDiscoverySession(
+      base::Bind(&BluetoothChromeOSTest::DiscoverySessionCallback,
+                 base::Unretained(this)),
+      base::Bind(&BluetoothChromeOSTest::ErrorCallback,
+                 base::Unretained(this)));
+
+  // Validate running adapter state.
+  EXPECT_NE("", adapter_->GetAddress());
+  EXPECT_NE("", adapter_->GetName());
+  EXPECT_TRUE(adapter_->IsInitialized());
+  EXPECT_TRUE(adapter_->IsPresent());
+  EXPECT_TRUE(adapter_->IsPowered());
+  EXPECT_TRUE(adapter_->IsDiscoverable());
+  EXPECT_TRUE(adapter_->IsDiscovering());
+  EXPECT_NE(dbus::ObjectPath(""), static_cast<BluetoothAdapterChromeOS*>(
+                                      adapter_.get())->object_path());
+
+  // OnDBusThreadManagerShutdown
+  static_cast<BluetoothAdapterChromeOS*>(adapter_.get())
+      ->OnDBusThreadManagerShutdown();
+
+  // Validate post shutdown state.
+  EXPECT_EQ("", adapter_->GetAddress());
+  EXPECT_EQ("", adapter_->GetName());
+  EXPECT_TRUE(adapter_->IsInitialized());
+  EXPECT_FALSE(adapter_->IsPresent());
+  EXPECT_FALSE(adapter_->IsPowered());
+  EXPECT_FALSE(adapter_->IsDiscoverable());
+  EXPECT_FALSE(adapter_->IsDiscovering());
+  EXPECT_EQ(dbus::ObjectPath(""), static_cast<BluetoothAdapterChromeOS*>(
+                                      adapter_.get())->object_path());

[armansito, 2015/01/16 21:57:18]

Probably also make a calls to the asynchronous methods here, e.g.
StartDiscoverySession should fail.

[scheib, 2015/01/16 22:28:27]

Done.

+}
+
 }  // namespace chromeos