Remove window destruction API and implementation

Eliminated the Destroy method from WindowManager and its C API, along with all platform-specific implementations. This simplifies the window management interface and removes unused or unsupported window destruction functionality.

Author: lijy91

src/capi/window_manager_c.cpp

@@ -182,16 +182,6 @@ native_window_t native_window_manager_get_current(void) {
   }
 }
 
-FFI_PLUGIN_EXPORT
-bool native_window_manager_destroy(native_window_id_t window_id) {
-  try {
-    auto& manager = WindowManager::GetInstance();
-    return manager.Destroy(window_id);
-  } catch (...) {
-    return false;
-  }
-}
-
 FFI_PLUGIN_EXPORT
 int native_window_manager_register_event_callback(native_window_event_callback_t callback,
                                                   void* user_data) {

src/capi/window_manager_c.h

@@ -82,14 +82,6 @@ native_window_list_t native_window_manager_get_all(void);
 FFI_PLUGIN_EXPORT
 native_window_t native_window_manager_get_current(void);
 
-/**
- * Destroy a window by its ID
- * @param window_id The window ID to destroy
- * @return true if window was found and destroyed, false otherwise
- */
-FFI_PLUGIN_EXPORT
-bool native_window_manager_destroy(native_window_id_t window_id);
-
 /**
  * Register a callback for window events
  * @param callback The callback function to register

src/platform/android/window_manager_android.cpp

@@ -106,15 +106,6 @@ std::shared_ptr<Window> WindowManager::GetCurrent() {
   return all.empty() ? nullptr : all.front();
 }
 
-bool WindowManager::Destroy(WindowId id) {
-  auto window = WindowRegistry::GetInstance().Get(id);
-  if (!window) {
-    return false;
-  }
-  WindowRegistry::GetInstance().Remove(id);
-  return true;
-}
-
 void WindowManager::SetWillShowHook(std::optional<WindowWillShowHook> hook) {
   // Empty implementation
 }

src/platform/ios/window_manager_ios.mm

@@ -37,17 +37,6 @@
   return nullptr;
 }
 
-bool WindowManager::Destroy(WindowId id) {
-  auto window = WindowRegistry::GetInstance().Get(id);
-  if (!window) {
-    return false;
-  }
-
-  WindowRegistry::GetInstance().Remove(id);
-
-  return true;
-}
-
 void WindowManager::SetWillShowHook(std::optional<WindowWillShowHook> hook) {
   // Empty implementation
 }

src/platform/linux/window_manager_linux.cpp

@@ -337,16 +337,6 @@ std::shared_ptr<Window> WindowManager::GetCurrent() {
   return nullptr;
 }
 
-bool WindowManager::Destroy(WindowId id) {
-  auto window = WindowRegistry::GetInstance().Get(id);
-  if (!window) {
-    return false;
-  }
-  // TODO: Implement proper GTK window destruction
-  WindowRegistry::GetInstance().Remove(id);
-  return true;
-}
-
 void WindowManager::SetWillShowHook(std::optional<WindowWillShowHook> hook) {
   pimpl_->will_show_hook_ = std::move(hook);
   if (pimpl_->will_show_hook_) {

src/platform/macos/window_manager_macos.mm

@@ -257,24 +257,6 @@ - (void)windowWillClose:(NSNotification*)notification {
   StopEventListening();
 }
 
-// Destroy a window by its ID. Returns true if window was destroyed.
-bool WindowManager::Destroy(WindowId id) {
-  auto window = WindowRegistry::GetInstance().Get(id);
-  if (!window) {
-    return false;
-  }
-  NSArray* ns_windows = [[NSApplication sharedApplication] windows];
-  for (NSWindow* ns_window in ns_windows) {
-    if ([ns_window windowNumber] == id) {
-      [ns_window close];
-      WindowRegistry::GetInstance().Remove(id);
-      return true;
-    }
-  }
-  WindowRegistry::GetInstance().Remove(id);
-  return false;
-}
-
 std::shared_ptr<Window> WindowManager::Get(WindowId id) {
   // First check if it's already in the registry
   auto window = WindowRegistry::GetInstance().Get(id);

src/platform/ohos/window_manager_ohos.cpp

@@ -102,15 +102,6 @@ std::shared_ptr<Window> WindowManager::GetCurrent() {
   return all.empty() ? nullptr : all.front();
 }
 
-bool WindowManager::Destroy(WindowId id) {
-  auto window = WindowRegistry::GetInstance().Get(id);
-  if (!window) {
-    return false;
-  }
-  WindowRegistry::GetInstance().Remove(id);
-  return true;
-}
-
 void WindowManager::SetWillShowHook(std::optional<WindowWillShowHook> hook) {
   // Empty implementation
 }

src/platform/windows/window_manager_windows.cpp

@@ -359,20 +359,6 @@ WindowManager::~WindowManager() {
   StopEventListening();
 }
 
-// Destroy a window by its ID. Returns true if window was destroyed.
-bool WindowManager::Destroy(WindowId id) {
-  auto window = WindowRegistry::GetInstance().Get(id);
-  if (!window) {
-    return false;
-  }
-  HWND hwnd = static_cast<HWND>(window->GetNativeObject());
-  if (IsWindow(hwnd)) {
-    DestroyWindow(hwnd);
-  }
-  WindowRegistry::GetInstance().Remove(id);
-  return true;
-}
-
 std::shared_ptr<Window> WindowManager::Get(WindowId id) {
   // First try to get from registry
   auto window = WindowRegistry::GetInstance().Get(id);

src/window_manager.h

@@ -118,26 +118,6 @@ class WindowManager : public EventEmitter<WindowEvent> {
    */
   std::shared_ptr<Window> GetCurrent();
 
-  /**
-   * @brief Destroy a window by its ID
-   *
-   * Removes the specified window from the registry and destroys it.
-   * This will close the window and free its resources.
-   * Any remaining shared_ptr references to the window will become invalid after
-   * the window is destroyed.
-   *
-   * @param id The unique identifier of the window to destroy
-   * @return true if the window was found and destroyed, false if window was not found
-   *
-   * @code
-   * WindowId window_id = some_window->GetId();
-   * bool success = WindowManager::GetInstance().Destroy(window_id);
-   * if (success) {
-   *     std::cout << "Window destroyed successfully" << std::endl;
-   * }
-   * @endcode
-   */
-  bool Destroy(WindowId id);
 
   /**
    * Hooks invoked before native window show/hide operations (e.g., via swizzling).