Make BluetoothAdapterBlueZ::Init early out if shutdown

device/bluetooth/bluez/bluetooth_adapter_bluez.cc

Index: device/bluetooth/bluez/bluetooth_adapter_bluez.cc
diff --git a/device/bluetooth/bluez/bluetooth_adapter_bluez.cc b/device/bluetooth/bluez/bluetooth_adapter_bluez.cc
index 5dd44416a7c794d914733e7bfbac7e29418aea5a..0ffd4f968aca411cd541f5e0076a45e08e2c7645 100644
--- a/device/bluetooth/bluez/bluetooth_adapter_bluez.cc
+++ b/device/bluetooth/bluez/bluetooth_adapter_bluez.cc
@@ -208,6 +208,10 @@ BluetoothAdapterBlueZ::BluetoothAdapterBlueZ(const InitCallback& init_callback)
 }
 
 void BluetoothAdapterBlueZ::Init() {
+  // We've been shutdown before Init() was run. Do nothing.
+  if (dbus_is_shutdown_)

[sky, 2016/08/25 17:15:09]

I don't know if we need to set initialized_ and run init_callback_ as happens on
218/219 in this case. One option is to combine the conditionals, by that I mean
move this into 217. I'm not familiar enough with the code to know which is
preferable.

[Rahul Chaturvedi, 2016/08/25 20:21:59]

In case a test is waiting on the adapter to get ready (which plenty do), they'd
time out. Even if they were to fail, we'd rather they fail than time out. So
this should be moved down to 217,
if (dbus_is_shutdown_ ||
!bluez::BluezDBusManager::Get()->IsObjectManagerSupported()) {

[sky, 2016/08/25 20:27:27]

Done.

+    return;
+
   // If the platform doesn't support Object Manager then Bluez 5 is probably
   // not present. In this case we just return without initializing anything.
   if (!bluez::BluezDBusManager::Get()->IsObjectManagerSupported()) {