arc: bluetooth: Don't unset bluetooth_adapter_

components/arc/bluetooth/arc_bluetooth_bridge.cc

 // Copyright 2016 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 "components/arc/bluetooth/arc_bluetooth_bridge.h"
 
 #include <fcntl.h>
 #include <stddef.h>
 
 #include <iomanip>
@@ skipping 84 matching lines @@
   mojom::BluetoothInstance* bluetooth_instance =
       arc_bridge_service()->bluetooth()->instance();
   if (!bluetooth_instance) {
     LOG(ERROR) << "OnBluetoothInstanceReady called, "
                << "but no bluetooth instance found";
     return;
   }
   bluetooth_instance->Init(binding_.CreateInterfacePtrAndBind());
 }
 
 void ArcBluetoothBridge::AdapterPresentChanged(BluetoothAdapter* adapter,

[rkc, 2016/07/22 19:59:48]

Why is this override necessary at all?
If the adapter becomes available again, we won't get that notification at all
with this code.

[puthik_chromium, 2016/07/22 22:46:13]

Done.

                                                bool present) {
   // If the adapter goes away, remove ourselves as an observer.
   if (!present && adapter == bluetooth_adapter_) {
     adapter->RemoveObserver(this);
-    bluetooth_adapter_ = nullptr;

[Luis Héctor Chávez, 2016/07/22 01:53:24]

is it guaranteed that the adapter (reference) will never change? if so, nit:

if (present || adapter != bluetooth_adapter_)
  return;
adapter->RemoveObserver(this);

   }
 }
 
 void ArcBluetoothBridge::AdapterPoweredChanged(BluetoothAdapter* adapter,
                                                bool powered) {
   if (!HasBluetoothInstance())
     return;
 
   // TODO(smbarber): Invoke EnableAdapter or DisableAdapter via ARC bridge
   // service.
@@ skipping 1233 matching lines @@
     uint32_t version_need) const {
   uint32_t version = arc_bridge_service()->bluetooth()->version();
   if (version >= version_need)
     return true;
   LOG(WARNING) << "Bluetooth instance is too old (version " << version
                << ") need version " << version_need;
   return false;
 }
 
 }  // namespace arc