Adding support for service scan on OS X

device/bluetooth/test/mock_bluetooth_cbperipheral_mac.mm

 // Copyright 2015 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 "device/bluetooth/test/mock_bluetooth_cbperipheral_mac.h"
 
 #include "base/mac/foundation_util.h"
 #include "base/mac/scoped_nsobject.h"
-#include "device/bluetooth/test/bluetooth_test.h"
+#include "device/bluetooth/test/bluetooth_test_mac.h"
+#include "device/bluetooth/test/mock_bluetooth_cbservice_mac.h"
 
 using base::mac::ObjCCast;
 using base::scoped_nsobject;
 
 @interface MockCBPeripheral () {
   scoped_nsobject<NSUUID> _identifier;
   scoped_nsobject<NSString> _name;
+  id<CBPeripheralDelegate> _delegate;
+  NSMutableArray* _services;

[ortuno, 2016/05/06 16:03:34]

How does this array get destroyed?

[jlebel, 2016/05/07 00:16:12]

Done.

 }
 
 @end
 
 @implementation MockCBPeripheral
 
 @synthesize state = _state;
+@synthesize delegate = _delegate;

[ortuno, 2016/05/06 16:03:34]

Why do you need to manually synthesize these, I thought the default was
_propertyName?
https://developer.apple.com/library/mac/documentation/Cocoa/Conceptual/Progra...

[jlebel, 2016/05/07 00:16:12]

Unfortunately Chrome is not using ARC yet. Therefore synthesize has to be done
explicitly.

[ortuno, 2016/05/09 19:54:01]

Ah. I see.

+@synthesize bluetoothTestMac = _bluetoothTestMac;
 
 - (instancetype)init {
   [self doesNotRecognizeSelector:_cmd];
   return self;
 }
 
 - (instancetype)initWithUTF8StringIdentifier:(const char*)utf8Identifier {
   scoped_nsobject<NSUUID> identifier(
       [[NSUUID alloc] initWithUUIDString:@(utf8Identifier)]);
   return [self initWithIdentifier:identifier name:nil];
 }
 
 - (instancetype)initWithIdentifier:(NSUUID*)identifier {
   return [self initWithIdentifier:identifier name:nil];
 }
 
 - (instancetype)initWithIdentifier:(NSUUID*)identifier name:(NSString*)name {
   self = [super init];
   if (self) {
+    _services = [[NSMutableArray alloc] init];
     _identifier.reset([identifier retain]);
     if (name) {
       _name.reset([name retain]);
     } else {
       _name.reset(
           [@(device::BluetoothTestBase::kTestDeviceName.c_str()) retain]);
     }
     _state = CBPeripheralStateDisconnected;
   }
   return self;
@@ skipping 12 matching lines @@
       [aClass isSubclassOfClass:[CBPeripheral class]]) {
     return YES;
   }
   return [super isKindOfClass:aClass];
 }
 
 - (void)setState:(CBPeripheralState)state {
   _state = state;
 }
 
+- (void)discoverServices:(NSArray*)serviceUUIDs {
+  if (_bluetoothTestMac) {
+    _bluetoothTestMac->OnFakeBluetoothServiceDiscovery();
+  }
+}
+
+- (void)removeAllServices {
+  [_services removeAllObjects];
+}
+
+- (void)addServices:(NSArray*)services {
+  for (CBUUID* uuid in services) {
+    MockCBService* service =

[ortuno, 2016/05/06 16:03:34]

Who manages the lifetime of this object? Does it's ownership get transferred to
the _services array?

[jlebel, 2016/05/07 00:16:12]

Done.

+        [[MockCBService alloc] initWithCBUUID:uuid primary:YES];
+    [_services addObject:service.service];
+  }
+}
+
+- (void)didDiscoverWithError:(NSError*)error {
+  [_delegate peripheral:self.peripheral didDiscoverServices:error];

[ortuno, 2016/05/06 16:03:34]

What is this called? I've never seen this pattern in Objective-C and I can't
find any docs about it. No need to change the CL I'm just curious what this is.

[jlebel, 2016/05/07 00:16:12]

What pattern are you talking about? "self.peripheral"?

[ortuno, 2016/05/09 19:54:01]

I guess I'm not sure what the second and third parts mean. I usually see the
following:

[object function:parameter]

But here I see:

[object name:parameter function:parameter]

+}
+
+- (void)removeServiceWithCBUUID:(CBUUID*)uuid {
+  CBService* serviceToRemove = nil;
+  for (MockCBService* service in _services) {
+    if ([service.UUID isEqual:uuid]) {
+      serviceToRemove = service.service;
+      break;
+    }
+  }
+  NSAssert(serviceToRemove, @"Unknown service to remove %@", uuid);
+  [_services removeObject:serviceToRemove];
+#pragma clang diagnostic push

[ortuno, 2016/05/06 16:03:34]

Can you comment why you need this?

[jlebel, 2016/05/07 00:16:12]

Done.

+#pragma clang diagnostic ignored "-Wpartial-availability"
+  [_delegate peripheral:self.peripheral didModifyServices:@[ serviceToRemove ]];
+#pragma clang diagnostic pop
+}
+
 - (NSUUID*)identifier {
   return _identifier.get();
 }
 
 - (NSString*)name {
   return _name.get();
 }
 
+- (NSArray*)services {
+  return _services;
+}
+
 - (CBPeripheral*)peripheral {
   return ObjCCast<CBPeripheral>(self);
 }
 
 @end