bluetooth: Shutdown BluetoothAdapter before DBus on ChromeOS.

device/bluetooth/bluetooth_adapter.h

 // Copyright (c) 2012 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 DEVICE_BLUETOOTH_BLUETOOTH_ADAPTER_H_
 #define DEVICE_BLUETOOTH_BLUETOOTH_ADAPTER_H_
 
 #include <list>
 #include <map>
 #include <set>
@@ skipping 170 matching lines @@
 
   // Returns a weak pointer to a new adapter.  For platforms with asynchronous
   // initialization, the returned adapter will run the |init_callback| once
   // asynchronous initialization is complete.
   // Caution: The returned pointer also transfers ownership of the adapter.  The
   // caller is expected to call |AddRef()| on the returned pointer, typically by
   // storing it into a |scoped_refptr|.
   static base::WeakPtr<BluetoothAdapter> CreateAdapter(
       const InitCallback& init_callback);
 
+#if defined(OS_CHROMEOS)
+  // Shutdown the adapter, ending reliance DBusThreadManager, and resulting in
+  // the adapter's |IsPresent| returning false.
+  virtual void OnDBusThreadManagerShutdown();

[stevenjb, 2015/01/14 23:44:14]

nit: We could make this a pure virtual and eliminate the need for a
NOTIMPLEMENTED method in the .cc file (and, ensure that it gets implemented on a
chromeos build).

[scheib, 2015/01/15 00:03:40]

Acknowledged.

+#endif

[armansito, 2015/01/14 23:49:47]

I wouldn't even add this method to BluetoothAdapter. You can just directly
static_cast the adapter to BluetoothAdapterChromeOS and call its shutdown method
from BluetoothAdapterFactory.

[scheib, 2015/01/15 00:03:40]

Done.

+
   // Returns a weak pointer to an existing adapter for testing purposes only.
   base::WeakPtr<BluetoothAdapter> GetWeakPtrForTesting();
 
   // Adds and removes observers for events on this bluetooth adapter. If
   // monitoring multiple adapters, check the |adapter| parameter of observer
   // methods to determine which adapter is issuing the event.
   virtual void AddObserver(BluetoothAdapter::Observer* observer) = 0;
   virtual void RemoveObserver(
       BluetoothAdapter::Observer* observer) = 0;
 
@@ skipping 229 matching lines @@
 // Trait for RefCountedThreadSafe that deletes BluetoothAdapter.
 struct BluetoothAdapterDeleter {
   static void Destruct(const BluetoothAdapter* adapter) {
     adapter->DeleteOnCorrectThread();
   }
 };
 
 }  // namespace device
 
 #endif  // DEVICE_BLUETOOTH_BLUETOOTH_ADAPTER_H_