Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(964)

Unified Diff: device/bluetooth/bluetooth_adapter_mac.mm

Issue 12929003: Implemented BluetoothAdapterMac::AddDevices(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Reverted BluetoothDevice interface. Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: device/bluetooth/bluetooth_adapter_mac.mm
diff --git a/device/bluetooth/bluetooth_adapter_mac.mm b/device/bluetooth/bluetooth_adapter_mac.mm
index 5d964ccbf9f31fa7d3a9fe2329d61d9b048aef20..c7d0eb5687fe05ac0972fa1aac9d21d12702a08c 100644
--- a/device/bluetooth/bluetooth_adapter_mac.mm
+++ b/device/bluetooth/bluetooth_adapter_mac.mm
@@ -4,28 +4,35 @@
#include "device/bluetooth/bluetooth_adapter_mac.h"
-#include <IOBluetooth/objc/IOBluetoothHostController.h>
+#import <IOBluetooth/objc/IOBluetoothDevice.h>
+#import <IOBluetooth/objc/IOBluetoothHostController.h>
#include <string>
#include "base/bind.h"
#include "base/compiler_specific.h"
+#include "base/hash_tables.h"
#include "base/location.h"
#include "base/sequenced_task_runner.h"
#include "base/single_thread_task_runner.h"
#include "base/thread_task_runner_handle.h"
#include "base/time.h"
#include "base/strings/sys_string_conversions.h"
+#include "device/bluetooth/bluetooth_device_mac.h"
// Replicate specific 10.7 SDK declarations for building with prior SDKs.
#if !defined(MAC_OS_X_VERSION_10_7) || \
MAC_OS_X_VERSION_MAX_ALLOWED < MAC_OS_X_VERSION_10_7
@interface IOBluetoothHostController (LionSDKDeclarations)
-- (NSString *)nameAsString;
+- (NSString*)nameAsString;
- (BluetoothHCIPowerState)powerState;
@end
+@interface IOBluetoothDevice (LionSDKDeclarations)
+- (NSString*)addressString;
+@end
+
#endif // MAC_OS_X_VERSION_10_7
namespace {
@@ -133,6 +140,10 @@ void BluetoothAdapterMac::PollAdapter() {
AdapterPoweredChanged(this, powered_));
}
+ NSArray* paired_devices = [IOBluetoothDevice pairedDevices];
+ AddDevices(paired_devices);
+ RemoveUnpairedDevices(paired_devices);
+
ui_task_runner_->PostDelayedTask(
FROM_HERE,
base::Bind(&BluetoothAdapterMac::PollAdapter,
@@ -140,4 +151,53 @@ void BluetoothAdapterMac::PollAdapter() {
base::TimeDelta::FromMilliseconds(kPollIntervalMs));
}
+void BluetoothAdapterMac::AddDevices(NSArray* devices) {
+ for (IOBluetoothDevice* device in devices) {
+ std::string device_address =
+ base::SysNSStringToUTF8([device addressString]);
+ DevicesMap::iterator found_device_iter = devices_.find(device_address);
+
+ if (found_device_iter == devices_.end()) {
+ devices_[device_address] = new BluetoothDeviceMac(device);
+ FOR_EACH_OBSERVER(BluetoothAdapter::Observer, observers_,
+ DeviceAdded(this, devices_[device_address]));
+ continue;
+ }
+ BluetoothDeviceMac* device_mac =
+ static_cast<BluetoothDeviceMac*>(found_device_iter->second);
+ if (device_mac->device_fingerprint() !=
+ BluetoothDeviceMac::ComputeDeviceFingerprint(device)) {
+ devices_[device_address] = new BluetoothDeviceMac(device);
+ FOR_EACH_OBSERVER(BluetoothAdapter::Observer, observers_,
+ DeviceChanged(this, devices_[device_address]));
+ delete device_mac;
+ }
+ }
+}
+
+void BluetoothAdapterMac::RemoveUnpairedDevices(NSArray* paired_devices) {
+ base::hash_set<std::string> paired_device_address_list;
+ for (IOBluetoothDevice* device in paired_devices) {
+ paired_device_address_list.insert(
+ base::SysNSStringToUTF8([device addressString]));
+ }
+
+ DevicesMap::iterator iter = devices_.begin();
+ while (iter != devices_.end()) {
+ DevicesMap::iterator device_iter = iter;
+ ++iter;
+
+ if (paired_device_address_list.find(device_iter->first) !=
+ paired_device_address_list.end())
+ continue;
+
+ if (device_iter->second->IsPaired()) {
+ FOR_EACH_OBSERVER(BluetoothAdapter::Observer, observers_,
+ DeviceRemoved(this, device_iter->second));
+ delete device_iter->second;
+ devices_.erase(device_iter);
+ }
+ }
+}
+
} // namespace device

Powered by Google App Engine
This is Rietveld 408576698