Clean up Bluetooth API implementation.

chrome/browser/extensions/api/bluetooth/bluetooth_api.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_api.h"
 
 #include <string>
 
 #include "base/bind_helpers.h"
 #include "base/lazy_instance.h"
@@ skipping 83 matching lines @@
 void BluetoothAPI::OnListenerRemoved(const EventListenerInfo& details) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   if (event_router()->IsBluetoothSupported())
     event_router()->OnListenerRemoved();
 }
 
 namespace api {
 
 BluetoothGetAdapterStateFunction::~BluetoothGetAdapterStateFunction() {}
 
-bool BluetoothGetAdapterStateFunction::DoWork(
+void BluetoothGetAdapterStateFunction::DoWork(
     scoped_refptr<BluetoothAdapter> adapter) {
   bluetooth::AdapterState state;
   PopulateAdapterState(*adapter.get(), &state);
   results_ = bluetooth::GetAdapterState::Results::Create(state);
   SendResponse(true);
-  return true;
 }
 
 BluetoothGetDevicesFunction::~BluetoothGetDevicesFunction() {}
 
-bool BluetoothGetDevicesFunction::DoWork(
+void BluetoothGetDevicesFunction::DoWork(
     scoped_refptr<BluetoothAdapter> adapter) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
 
   base::ListValue* device_list = new base::ListValue;
   SetResult(device_list);
 
   BluetoothAdapter::DeviceList devices = adapter->GetDevices();
   for (BluetoothAdapter::DeviceList::const_iterator iter = devices.begin();
        iter != devices.end();
        ++iter) {
     const BluetoothDevice* device = *iter;
     DCHECK(device);
 
     bluetooth::Device extension_device;
     bluetooth::BluetoothDeviceToApiDevice(*device, &extension_device);
 
     device_list->Append(extension_device.ToValue().release());
   }
 
   SendResponse(true);
-
-  return true;
 }
 
 BluetoothGetDeviceFunction::~BluetoothGetDeviceFunction() {}
 
-bool BluetoothGetDeviceFunction::DoWork(
+void BluetoothGetDeviceFunction::DoWork(
     scoped_refptr<BluetoothAdapter> adapter) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
 
   scoped_ptr<GetDevice::Params> params(GetDevice::Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params.get() != NULL);
 
   BluetoothDevice* device = adapter->GetDevice(params->device_address);
   if (device) {
     bluetooth::Device extension_device;
     bluetooth::BluetoothDeviceToApiDevice(*device, &extension_device);
     SetResult(extension_device.ToValue().release());
     SendResponse(true);
   } else {
     SetError(kInvalidDevice);
     SendResponse(false);
   }
-
-  return false;
 }
 
 void BluetoothStartDiscoveryFunction::OnSuccessCallback() {
   SendResponse(true);
 }
 
 void BluetoothStartDiscoveryFunction::OnErrorCallback() {
   SetError(kStartDiscoveryFailed);
   SendResponse(false);
 }
 
-bool BluetoothStartDiscoveryFunction::DoWork(
+void BluetoothStartDiscoveryFunction::DoWork(
     scoped_refptr<BluetoothAdapter> adapter) {
   GetEventRouter(browser_context())->StartDiscoverySession(
       adapter,
       extension_id(),
       base::Bind(&BluetoothStartDiscoveryFunction::OnSuccessCallback, this),
       base::Bind(&BluetoothStartDiscoveryFunction::OnErrorCallback, this));
-
-  return true;
 }
 
 void BluetoothStopDiscoveryFunction::OnSuccessCallback() {
   SendResponse(true);
 }
 
 void BluetoothStopDiscoveryFunction::OnErrorCallback() {
   SetError(kStopDiscoveryFailed);
   SendResponse(false);
 }
 
-bool BluetoothStopDiscoveryFunction::DoWork(
+void BluetoothStopDiscoveryFunction::DoWork(
     scoped_refptr<BluetoothAdapter> adapter) {
   GetEventRouter(browser_context())->StopDiscoverySession(
       adapter,
       extension_id(),
       base::Bind(&BluetoothStopDiscoveryFunction::OnSuccessCallback, this),
       base::Bind(&BluetoothStopDiscoveryFunction::OnErrorCallback, this));
-
-  return true;
 }
 
 }  // namespace api
 }  // namespace extensions