Followup project-chip#27318 - Fix review comments that came in after …

…landing
vivien-apple · Jul 3, 2023 · 5501901 · 5501901
1 parent b3002c6
commit 5501901
Show file tree

Hide file tree

Showing 9 changed files with 427 additions and 72 deletions.
diff --git a/examples/darwin-framework-tool/commands/discover/DiscoverCommissionablesCommand.mm b/examples/darwin-framework-tool/commands/discover/DiscoverCommissionablesCommand.mm
@@ -21,31 +21,32 @@
 auto gDispatchQueue = dispatch_queue_create("com.chip.discover", DISPATCH_QUEUE_SERIAL);
 
 @interface DeviceScannerDelegate : NSObject <MTRCommissionableBrowserDelegate>
-- (void)didDiscoverCommissionable:(MTRCommissionableBrowserResult *)device;
-- (void)commissionableUnavailable:(MTRCommissionableBrowserResult *)device;
+- (void)controller:(MTRDeviceController *)controller didFindCommissionableDevice:(MTRCommissionableBrowserResult *)device;
+- (void)controller:(MTRDeviceController *)controller didRemoveCommissionableDevice:(MTRCommissionableBrowserResult *)device;
 @end
 
 @implementation DeviceScannerDelegate
-- (void)didDiscoverCommissionable:(MTRCommissionableBrowserResult *)device
+- (void)controller:(MTRDeviceController *)controller didFindCommissionableDevice:(MTRCommissionableBrowserResult *)device
 {
-    auto serviceName = device.serviceName;
-    auto vendorId = device.vendorId;
-    auto productId = device.productId;
+    auto instanceName = device.instanceName;
+    auto vendorId = device.vendorID;
+    auto productId = device.productID;
     auto discriminator = device.discriminator;
     [gDiscoveredDevices addObject:device];
 
-    NSLog(@"Found Device (%@) with discriminator: %@ (vendor: %@, product: %@)", serviceName, discriminator, vendorId, productId);
+    NSLog(@"Found Device (%@) with discriminator: %@ (vendor: %@, product: %@)", instanceName, discriminator, vendorId, productId);
 }
 
-- (void)commissionableUnavailable:(MTRCommissionableBrowserResult *)device
+- (void)controller:(MTRDeviceController *)controller didRemoveCommissionableDevice:(MTRCommissionableBrowserResult *)device
 {
-    auto serviceName = device.serviceName;
-    auto vendorId = device.vendorId;
-    auto productId = device.productId;
+    auto instanceName = device.instanceName;
+    auto vendorId = device.vendorID;
+    auto productId = device.productID;
     auto discriminator = device.discriminator;
     [gDiscoveredDevices removeObjectIdenticalTo:device];
 
-    NSLog(@"Removed Device (%@) with discriminator: %@ (vendor: %@, product: %@)", serviceName, discriminator, vendorId, productId);
+    NSLog(
+        @"Removed Device (%@) with discriminator: %@ (vendor: %@, product: %@)", instanceName, discriminator, vendorId, productId);
 }
 @end
 
@@ -58,7 +59,7 @@ - (void)commissionableUnavailable:(MTRCommissionableBrowserResult *)device
     });
 
     auto delegate = [[DeviceScannerDelegate alloc] init];
-    auto success = [CurrentCommissioner() startScan:delegate queue:gDispatchQueue];
+    auto success = [CurrentCommissioner() startBrowseForCommissionables:delegate queue:gDispatchQueue];
     VerifyOrReturnError(success, CHIP_ERROR_INTERNAL);
 
     SetCommandExitStatus(CHIP_NO_ERROR);
@@ -69,7 +70,7 @@ - (void)commissionableUnavailable:(MTRCommissionableBrowserResult *)device
 {
     VerifyOrReturnError(IsInteractive(), CHIP_ERROR_INCORRECT_STATE);
 
-    auto success = [CurrentCommissioner() stopScan];
+    auto success = [CurrentCommissioner() stopBrowseForCommissionables];
     VerifyOrReturnError(success, CHIP_ERROR_INTERNAL);
 
     SetCommandExitStatus(CHIP_NO_ERROR);
@@ -86,13 +87,13 @@ - (void)commissionableUnavailable:(MTRCommissionableBrowserResult *)device
 
         uint16_t index = 0;
         for (id device in gDiscoveredDevices) {
-            auto serviceName = [device serviceName];
-            auto vendorId = [device vendorId];
-            auto productId = [device productId];
+            auto instanceName = [device instanceName];
+            auto vendorId = [device vendorID];
+            auto productId = [device productID];
             auto discriminator = [device discriminator];
 
-            NSLog(
-                @"\t %u %@ - Discriminator: %@ - Vendor: %@ - Product: %@", index, serviceName, discriminator, vendorId, productId);
+            NSLog(@"\t %u %@ - Discriminator: %@ - Vendor: %@ - Product: %@", index, instanceName, discriminator, vendorId,
+                productId);
 
             index++;
         }

diff --git a/src/darwin/Framework/CHIP/MTRCommissionableBrowser.h b/src/darwin/Framework/CHIP/MTRCommissionableBrowser.h
@@ -21,13 +21,25 @@
 NS_ASSUME_NONNULL_BEGIN
 
 @protocol MTRCommissionableBrowserDelegate;
+@class MTRDeviceController;
 
 MTR_HIDDEN
 @interface MTRCommissionableBrowser : NSObject
 - (instancetype)init NS_UNAVAILABLE;
 + (instancetype)new NS_UNAVAILABLE;
-- (instancetype)initWithDelegate:(id<MTRCommissionableBrowserDelegate>)delegate queue:(dispatch_queue_t)queue;
+- (instancetype)initWithDelegate:(id<MTRCommissionableBrowserDelegate>)delegate
+                      controller:(MTRDeviceController *)controller
+                           queue:(dispatch_queue_t)queue;
+/**
+ * Start browsing the available networks (e.g IP, BLE) for commissionable nodes.
+ *
+ * If a browse is already ongoing this will not start a new browse and the return value will be NO.
+ */
 - (BOOL)start;
+
+/**
+ * Stop browsing the network for commissionable nodes.
+ */
 - (BOOL)stop;
 @end