Revert "[nrf fromtree] Add factory reset event (#37993)"

This reverts commit f2bba3a.

Author: adigie

src/app/server/Server.cpp

@@ -586,22 +586,8 @@ void Server::GenerateShutDownEvent()
     PlatformMgr().ScheduleWork([](intptr_t) { PlatformMgr().HandleServerShuttingDown(); });
 }
 
-void Server::PostFactoryResetEvent()
-{
-    DeviceLayer::ChipDeviceEvent event;
-    event.Type = DeviceLayer::DeviceEventType::kFactoryReset;
-
-    CHIP_ERROR error = DeviceLayer::PlatformMgr().PostEvent(&event);
-    if (error != CHIP_NO_ERROR)
-    {
-        ChipLogError(AppServer, "Posting kFactoryReset event failed with %" CHIP_ERROR_FORMAT, error.Format());
-    }
-}
-
 void Server::ScheduleFactoryReset()
 {
-    PostFactoryResetEvent();
-
     PlatformMgr().ScheduleWork([](intptr_t) {
         // Delete all fabrics and emit Leave event.
         GetInstance().GetFabricTable().DeleteAllFabrics();

src/app/server/Server.h

@@ -456,8 +456,6 @@ class Server
     void OnPlatformEvent(const DeviceLayer::ChipDeviceEvent & event);
     void CheckServerReadyEvent();
 
-    void PostFactoryResetEvent();
-
     static void OnPlatformEventWrapper(const DeviceLayer::ChipDeviceEvent * event, intptr_t);
 
 #if CHIP_CONFIG_PERSIST_SUBSCRIPTIONS

src/app/tests/BUILD.gn

@@ -217,7 +217,6 @@ chip_test_suite("tests") {
     "TestReadInteraction.cpp",
     "TestReportScheduler.cpp",
     "TestReportingEngine.cpp",
-    "TestServer.cpp",
     "TestStatusIB.cpp",
     "TestStatusResponseMessage.cpp",
     "TestTestEventTriggerDelegate.cpp",

src/app/tests/TestServer.cpp

@@ -255,11 +255,6 @@ enum PublicEventTypes
      * Signals that secure session is established.
      */
     kSecureSessionEstablished,
-
-    /**
-     * Signals that factory reset has started.
-     */
-    kFactoryReset,
 };
 
 /**

src/include/platform/CHIPDeviceEvent.h

@@ -22,10 +22,8 @@
 
 #pragma once
 
-#include <platform/CHIPDeviceLayer.h>
+#include <platform/PlatformManager.h>
 
-#include <algorithm>
-#include <list>
 #include <queue>
 
 namespace chip {
@@ -46,34 +44,11 @@ class PlatformManagerImpl final : public PlatformManager
 private:
     // ===== Methods that implement the PlatformManager abstract interface.
 
-    struct EventHandler
-    {
-        PlatformManager::EventHandlerFunct Handler;
-        intptr_t Arg;
-
-        bool operator==(const EventHandler & other) const { return Handler == other.Handler && Arg == other.Arg; }
-    };
-
     CHIP_ERROR _InitChipStack() { return CHIP_NO_ERROR; }
     void _Shutdown() {}
 
-    CHIP_ERROR _AddEventHandler(EventHandlerFunct handler, intptr_t arg = 0)
-    {
-        EventHandler eventHandler = { .Handler = handler, .Arg = arg };
-        if (std::find(mEventHandlers.begin(), mEventHandlers.end(), eventHandler) == mEventHandlers.end())
-        {
-            mEventHandlers.push_back(eventHandler);
-        }
-
-        return CHIP_NO_ERROR;
-    }
-
-    void _RemoveEventHandler(EventHandlerFunct handler, intptr_t arg = 0)
-    {
-        EventHandler eventHandler = { .Handler = handler, .Arg = arg };
-        mEventHandlers.remove(eventHandler);
-    }
-
+    CHIP_ERROR _AddEventHandler(EventHandlerFunct handler, intptr_t arg = 0) { return CHIP_ERROR_NOT_IMPLEMENTED; }
+    void _RemoveEventHandler(EventHandlerFunct handler, intptr_t arg = 0) {}
     void _HandleServerStarted() {}
     void _HandleServerShuttingDown() {}
 
@@ -133,26 +108,8 @@ class PlatformManagerImpl final : public PlatformManager
             event->CallWorkFunct.WorkFunct(event->CallWorkFunct.Arg);
             break;
 
-        default: {
-#if CHIP_DEVICE_CONFIG_ENABLE_CHIPOBLE
-            BLEMgr().OnPlatformEvent(event);
-#endif
-#if CHIP_DEVICE_CONFIG_ENABLE_THREAD
-            ThreadStackMgr().OnPlatformEvent(event);
-#endif
-            ConnectivityMgr().OnPlatformEvent(event);
-
-            if (!event->IsInternal())
-            {
-                // iterate over local copy in case handler unregisters itself
-                auto handlers = mEventHandlers;
-                for (auto & handler : handlers)
-                {
-                    handler.Handler(event, handler.Arg);
-                }
-            }
-        }
-        break;
+        default:
+            break;
         }
     }
 
@@ -178,7 +135,6 @@ class PlatformManagerImpl final : public PlatformManager
 
     bool mShouldRunEventLoop = true;
     std::queue<ChipDeviceEvent> mQueue;
-    std::list<EventHandler> mEventHandlers;
 };
 
 /**

src/platform/fake/PlatformManagerImpl.h

@@ -85,7 +85,3 @@ CONFIG_CHIP_ENABLE_READ_CLIENT=y
 CONFIG_CHIP_DEVICE_VENDOR_ID=65521
 CONFIG_CHIP_DEVICE_PRODUCT_ID=32768
 CONFIG_CHIP_PROJECT_CONFIG="main/include/CHIPProjectConfig.h"
-
-# Don't erase all settings as it deinitializes NVS/ZMS and causes issues with
-# testing Server::ScheduleFactoryReset
-CONFIG_CHIP_FACTORY_RESET_ERASE_SETTINGS=n