cros: Clean up TrayBluetoothHelper todos

ash/common/system/chromeos/bluetooth/tray_bluetooth_helper.cc

 // Copyright 2017 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 "ash/common/system/chromeos/bluetooth/tray_bluetooth_helper.h"
 
 #include "ash/common/system/tray/system_tray_controller.h"
 #include "ash/common/system/tray/system_tray_notifier.h"
 #include "ash/common/wm_shell.h"
 #include "ash/shell.h"
@@ skipping 42 matching lines @@
                  weak_ptr_factory_.GetWeakPtr()));
 }
 
 void TrayBluetoothHelper::InitializeOnAdapterReady(
     scoped_refptr<device::BluetoothAdapter> adapter) {
   adapter_ = adapter;
   CHECK(adapter_);
   adapter_->AddObserver(this);
 }
 
-void TrayBluetoothHelper::GetAvailableBluetoothDevices(
-    std::vector<BluetoothDeviceInfo>* list) {
-  device::BluetoothAdapter::DeviceList devices = adapter_->GetDevices();
-  for (device::BluetoothDevice* device : devices) {
+BluetoothDeviceList TrayBluetoothHelper::GetAvailableBluetoothDevices() const {
+  BluetoothDeviceList list;
+  for (device::BluetoothDevice* device : adapter_->GetDevices()) {
     BluetoothDeviceInfo info;
     info.address = device->GetAddress();
     info.display_name = device->GetNameForDisplay();
     info.connected = device->IsConnected();
     info.connecting = device->IsConnecting();
     info.paired = device->IsPaired();
     info.device_type = device->GetDeviceType();
-    list->push_back(info);
+    list.push_back(info);
   }
+  return list;
 }
 
 void TrayBluetoothHelper::StartBluetoothDiscovering() {
   if (HasBluetoothDiscoverySession()) {
     LOG(WARNING) << "Already have active Bluetooth device discovery session.";
     return;
   }
   VLOG(1) << "Requesting new Bluetooth device discovery session.";
   should_run_discovery_ = true;
   adapter_->StartDiscoverySession(
@@ skipping 93 matching lines @@
   // (e.g. the user dismissed the Bluetooth detailed view before the call
   // returned), don't claim the discovery session and let it clean up.
   if (!should_run_discovery_)
     return;
   VLOG(1) << "Claiming new Bluetooth device discovery session.";
   discovery_session_ = std::move(discovery_session);
   GetSystemTrayNotifier()->NotifyBluetoothDiscoveringChanged();
 }
 
 }  // namespace ash