address comments

Author: yunhanw-google

examples/chip-tool/commands/clusters/ClusterCommand.h

@@ -114,11 +114,7 @@ class ClusterCommand : public InteractionModelCommands, public ModelCommand, pub
         if ((path.mEndpointId == chip::kRootEndpointId) && (path.mClusterId == chip::app::Clusters::IcdManagement::Id) &&
             (path.mCommandId == chip::app::Clusters::IcdManagement::Commands::UnregisterClient::Id))
         {
-            CHIP_ERROR deleteEntryError = CHIPCommand::sICDClientStorage.DeleteEntry(chip::ScopedNodeId(mPeerNodeId, mFabricIndex));
-            if (deleteEntryError != CHIP_NO_ERROR)
-            {
-                ChipLogError(chipTool, "Failed to delete ICD entry: %s", chip::ErrorStr(deleteEntryError));
-            }
+            ModelCommand::ClearICDEntry(chip::ScopedNodeId(mPeerNodeId, mFabricIndex));
         }
     }
 

examples/chip-tool/commands/clusters/ModelCommand.cpp

@@ -76,6 +76,15 @@ void ModelCommand::Shutdown()
     CHIPCommand::Shutdown();
 }
 
+void ModelCommand::ClearICDEntry(const chip::ScopedNodeId & nodeId)
+{
+    CHIP_ERROR deleteEntryError = CHIPCommand::sICDClientStorage.DeleteEntry(nodeId);
+    if (deleteEntryError != CHIP_NO_ERROR)
+    {
+        ChipLogError(chipTool, "Failed to delete ICD entry: %" CHIP_ERROR_FORMAT, deleteEntryError.Format());
+    }
+}
+
 void ModelCommand::CheckPeerICDType()
 {
     if (mIsPeerLIT.HasValue())

examples/chip-tool/commands/clusters/ModelCommand.h

@@ -67,6 +67,8 @@ class ModelCommand : public CHIPCommand
 
     virtual CHIP_ERROR SendGroupCommand(chip::GroupId groupId, chip::FabricIndex fabricIndex) { return CHIP_ERROR_BAD_REQUEST; };
 
+    void ClearICDEntry(const chip::ScopedNodeId & nodeId) override;
+
     void Shutdown() override;
 
 protected:

examples/tv-casting-app/tv-casting-common/commands/clusters/ModelCommand.cpp

@@ -81,3 +81,8 @@ void ModelCommand::Shutdown()
     mOnDeviceConnectedCallback.Cancel();
     mOnDeviceConnectionFailureCallback.Cancel();
 }
+
+void ClearICDEntry(const chip::ScopedNodeId & nodeId);
+{
+    ChipLogError(chipTool, "ClearICDEntry is not implemented");
+}