Changed DefaultAdapter to RunCallbackOnAdapterReady function.

chrome/browser/extensions/api/bluetooth/bluetooth_event_router.cc

 // 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.
 
 #include "chrome/browser/extensions/api/bluetooth/bluetooth_event_router.h"
 
 #include <map>
 #include <string>
 
 #include "base/json/json_writer.h"
@@ skipping 23 matching lines @@
 }
 
 ExtensionBluetoothEventRouter::~ExtensionBluetoothEventRouter() {
   CHECK(socket_map_.size() == 0);
   if (adapter_) {
     adapter_->RemoveObserver(this);
     adapter_ = NULL;
   }
 }
 
-scoped_refptr<device::BluetoothAdapter>
-ExtensionBluetoothEventRouter::GetAdapter() {
-  if (adapter_)
-    return adapter_;
-
-  return device::BluetoothAdapterFactory::DefaultAdapter();
+void ExtensionBluetoothEventRouter::RunCallbackOnAdapterReady(
+    const device::BluetoothAdapterFactory::AdapterCallback& callback) {
+  device::BluetoothAdapterFactory::RunCallbackOnAdapterReady(callback);

[bryeung, 2013/01/08 22:24:27]

Since this is just a pass-through method now...why does it need to remain part
of the public API?

[youngki, 2013/01/09 15:12:03]

We could.. But then we cannot mock the bluetooth adapter in the unittest..
Without this I couldn't find a suitable way to inject the mock bluetooth adapter
to the unittest.

[bryeung, 2013/01/09 15:28:01]

Okay.

 }
 
 void ExtensionBluetoothEventRouter::OnListenerAdded() {
   num_event_listeners_++;
   InitializeAdapterIfNeeded();
 }
 
 void ExtensionBluetoothEventRouter::OnListenerRemoved() {
   num_event_listeners_--;
   CHECK(num_event_listeners_ >= 0);
@@ skipping 111 matching lines @@
 
   if (!send_discovery_events_)
     return;
 
   DispatchDeviceEvent(extensions::event_names::kBluetoothOnDeviceDiscovered,
                       *extension_device);
 }
 
 void ExtensionBluetoothEventRouter::InitializeAdapterIfNeeded() {
   if (!adapter_) {
-    adapter_ = GetAdapter();
-    adapter_->AddObserver(this);
+    RunCallbackOnAdapterReady(
+        base::Bind(&ExtensionBluetoothEventRouter::InitializeAdapter,
+                   base::Unretained(this)));
   }
 }
 
+bool ExtensionBluetoothEventRouter::InitializeAdapter(
+    scoped_refptr<device::BluetoothAdapter> adapter) {
+  adapter_ = adapter;
+  adapter_->AddObserver(this);
+  return true;
+}
+
 void ExtensionBluetoothEventRouter::MaybeReleaseAdapter() {
   if (adapter_ && num_event_listeners_ == 0) {
     adapter_->RemoveObserver(this);
     adapter_ = NULL;
   }
 }
 
 void ExtensionBluetoothEventRouter::DispatchAdapterStateEvent() {
   api::bluetooth::AdapterState state;
   PopulateAdapterState(*adapter_, &state);
 
   scoped_ptr<ListValue> args(new ListValue());
   args->Append(state.ToValue().release());
   scoped_ptr<Event> event(new Event(
       extensions::event_names::kBluetoothOnAdapterStateChanged,
       args.Pass()));
   ExtensionSystem::Get(profile_)->event_router()->BroadcastEvent(event.Pass());
 }
 
 }  // namespace extensions