Address review comments

Author: yufengwangca

examples/fabric-bridge-app/fabric-bridge-common/fabric-bridge-app.matter

@@ -21,10 +21,8 @@ assert(chip_build_tools)
 executable("fabric-bridge-app") {
   sources = [
     "${chip_root}/examples/fabric-bridge-app/fabric-bridge-common/include/CHIPProjectAppConfig.h",
-    "${chip_root}/examples/fabric-bridge-app/linux/ActionsServerCallbacks.cpp",
     "Device.cpp",
     "include/Device.h",
-    "include/main.h",
     "main.cpp",
   ]
 

examples/fabric-bridge-app/fabric-bridge-common/fabric-bridge-app.zap

@@ -72,58 +72,3 @@ void Device::SetName(const char * szName)
         HandleDeviceChange(this, kChanged_Name);
     }
 }
-
-void Device::SetLocation(std::string szLocation)
-{
-    bool changed = (mLocation.compare(szLocation) != 0);
-
-    mLocation = szLocation;
-
-    ChipLogProgress(DeviceLayer, "Device[%s]: Location=\"%s\"", mName, mLocation.c_str());
-
-    if (changed)
-    {
-        HandleDeviceChange(this, kChanged_Location);
-    }
-}
-
-EndpointListInfo::EndpointListInfo(uint16_t endpointListId, std::string name, EndpointListTypeEnum type)
-{
-    mEndpointListId = endpointListId;
-    mName           = name;
-    mType           = type;
-}
-
-EndpointListInfo::EndpointListInfo(uint16_t endpointListId, std::string name, EndpointListTypeEnum type,
-                                   chip::EndpointId endpointId)
-{
-    mEndpointListId = endpointListId;
-    mName           = name;
-    mType           = type;
-    mEndpoints.push_back(endpointId);
-}
-
-void EndpointListInfo::AddEndpointId(chip::EndpointId endpointId)
-{
-    mEndpoints.push_back(endpointId);
-}
-
-Room::Room(std::string name, uint16_t endpointListId, EndpointListTypeEnum type, bool isVisible)
-{
-    mName           = name;
-    mEndpointListId = endpointListId;
-    mType           = type;
-    mIsVisible      = isVisible;
-}
-
-Action::Action(uint16_t actionId, std::string name, ActionTypeEnum type, uint16_t endpointListId, uint16_t supportedCommands,
-               ActionStateEnum status, bool isVisible)
-{
-    mActionId          = actionId;
-    mName              = name;
-    mType              = type;
-    mEndpointListId    = endpointListId;
-    mSupportedCommands = supportedCommands;
-    mStatus            = status;
-    mIsVisible         = isVisible;
-}

examples/fabric-bridge-app/linux/ActionsServerCallbacks.cpp

@@ -67,66 +67,3 @@ class Device
     chip::EndpointId mParentEndpointId;
     std::string mZone;
 };
-
-class EndpointListInfo
-{
-public:
-    EndpointListInfo(uint16_t endpointListId, std::string name, chip::app::Clusters::Actions::EndpointListTypeEnum type);
-    EndpointListInfo(uint16_t endpointListId, std::string name, chip::app::Clusters::Actions::EndpointListTypeEnum type,
-                     chip::EndpointId endpointId);
-    void AddEndpointId(chip::EndpointId endpointId);
-    inline uint16_t GetEndpointListId() { return mEndpointListId; };
-    std::string GetName() { return mName; };
-    inline chip::app::Clusters::Actions::EndpointListTypeEnum GetType() { return mType; };
-    inline chip::EndpointId * GetEndpointListData() { return mEndpoints.data(); };
-    inline size_t GetEndpointListSize() { return mEndpoints.size(); };
-
-private:
-    uint16_t mEndpointListId = static_cast<uint16_t>(0);
-    std::string mName;
-    chip::app::Clusters::Actions::EndpointListTypeEnum mType = static_cast<chip::app::Clusters::Actions::EndpointListTypeEnum>(0);
-    std::vector<chip::EndpointId> mEndpoints;
-};
-
-class Room
-{
-public:
-    Room(std::string name, uint16_t endpointListId, chip::app::Clusters::Actions::EndpointListTypeEnum type, bool isVisible);
-    inline void setIsVisible(bool isVisible) { mIsVisible = isVisible; };
-    inline bool getIsVisible() { return mIsVisible; };
-    inline void setName(std::string name) { mName = name; };
-    inline std::string getName() { return mName; };
-    inline chip::app::Clusters::Actions::EndpointListTypeEnum getType() { return mType; };
-    inline uint16_t getEndpointListId() { return mEndpointListId; };
-
-private:
-    bool mIsVisible;
-    std::string mName;
-    uint16_t mEndpointListId;
-    chip::app::Clusters::Actions::EndpointListTypeEnum mType;
-};
-
-class Action
-{
-public:
-    Action(uint16_t actionId, std::string name, chip::app::Clusters::Actions::ActionTypeEnum type, uint16_t endpointListId,
-           uint16_t supportedCommands, chip::app::Clusters::Actions::ActionStateEnum status, bool isVisible);
-    inline void setName(std::string name) { mName = name; };
-    inline std::string getName() { return mName; };
-    inline chip::app::Clusters::Actions::ActionTypeEnum getType() { return mType; };
-    inline chip::app::Clusters::Actions::ActionStateEnum getStatus() { return mStatus; };
-    inline uint16_t getActionId() { return mActionId; };
-    inline uint16_t getEndpointListId() { return mEndpointListId; };
-    inline uint16_t getSupportedCommands() { return mSupportedCommands; };
-    inline void setIsVisible(bool isVisible) { mIsVisible = isVisible; };
-    inline bool getIsVisible() { return mIsVisible; };
-
-private:
-    std::string mName;
-    chip::app::Clusters::Actions::ActionTypeEnum mType;
-    chip::app::Clusters::Actions::ActionStateEnum mStatus;
-    uint16_t mActionId;
-    uint16_t mEndpointListId;
-    uint16_t mSupportedCommands;
-    bool mIsVisible;
-};