Stop using controllerID for internal XPC bookkeeping. (#28480)

We want to use controllerID for client-assigned controller IDs.
project-chip · Aug 2, 2023 · cfa8212 · cfa8212
1 parent 912339d
commit cfa8212
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/src/darwin/Framework/CHIP/MTRDeviceControllerOverXPC.mm b/src/darwin/Framework/CHIP/MTRDeviceControllerOverXPC.mm
@@ -179,7 +179,7 @@ - (void)fetchControllerIdWithQueue:(dispatch_queue_t)queue completion:(MTRFetchC
 
     dispatch_async(_workQueue, ^{
         dispatch_group_t group = dispatch_group_create();
-        if (!self.controllerID) {
+        if (!self.controllerXPCID) {
             dispatch_group_enter(group);
             [self.xpcConnection getProxyHandleWithCompletion:^(
                 dispatch_queue_t _Nonnull proxyQueue, MTRDeviceControllerXPCProxyHandle * _Nullable handle) {
@@ -188,7 +188,7 @@ - (void)fetchControllerIdWithQueue:(dispatch_queue_t)queue completion:(MTRFetchC
                         if (error) {
                             MTR_LOG_ERROR("Failed to fetch any shared remote controller");
                         } else {
-                            self.controllerID = controller;
+                            self.controllerXPCID = controller;
                             handleRetainer = handle;
                         }
                         dispatch_group_leave(group);
@@ -200,8 +200,8 @@ - (void)fetchControllerIdWithQueue:(dispatch_queue_t)queue completion:(MTRFetchC
             }];
         }
         dispatch_group_notify(group, queue, ^{
-            if (self.controllerID) {
-                completion(self.controllerID, handleRetainer, nil);
+            if (self.controllerXPCID) {
+                completion(self.controllerXPCID, handleRetainer, nil);
             } else {
                 completion(nil, nil, [NSError errorWithDomain:MTRErrorDomain code:MTRErrorCodeGeneralError userInfo:nil]);
             }
@@ -234,7 +234,7 @@ - (instancetype)initWithControllerID:(id)controllerID
                            workQueue:(dispatch_queue_t)queue
                        xpcConnection:(MTRDeviceControllerXPCConnection *)xpcConnection
 {
-    _controllerID = controllerID;
+    _controllerXPCID = controllerID;
     _workQueue = queue;
     _xpcConnection = xpcConnection;
     return self;

diff --git a/src/darwin/Framework/CHIP/MTRDeviceControllerOverXPC_Internal.h b/src/darwin/Framework/CHIP/MTRDeviceControllerOverXPC_Internal.h
@@ -25,7 +25,7 @@ typedef void (^MTRFetchControllerIDCompletion)(
 
 @interface MTRDeviceControllerOverXPC ()
 
-@property (nonatomic, readwrite, strong) id<NSCopying> _Nullable controllerID;
+@property (nonatomic, readwrite, strong) id<NSCopying> _Nullable controllerXPCID;
 @property (nonatomic, readonly, strong) dispatch_queue_t workQueue;
 @property (nonatomic, readonly, strong) MTRDeviceControllerXPCConnection * xpcConnection;
 

diff --git a/src/darwin/Framework/CHIP/MTRDeviceOverXPC.mm b/src/darwin/Framework/CHIP/MTRDeviceOverXPC.mm
@@ -46,7 +46,7 @@ - (instancetype)initWithControllerOverXPC:(MTRDeviceControllerOverXPC *)controll
                                  deviceID:(NSNumber *)deviceID
                             xpcConnection:(MTRDeviceControllerXPCConnection *)xpcConnection
 {
-    _controllerID = controllerOverXPC.controllerID;
+    _controllerID = controllerOverXPC.controllerXPCID;
     _controller = controllerOverXPC;
     _nodeID = deviceID;
     _xpcConnection = xpcConnection;