From 180c3e2fefae4d9ff173073e5cc5d59ab96ee729 Mon Sep 17 00:00:00 2001 From: Aliaksandr Kalenik Date: Mon, 4 Sep 2023 16:38:24 +0200 Subject: [PATCH] LibWeb: Remove unused BrowsingContext::close() Was replaced by close_top_level_traversable() --- .../Libraries/LibWeb/HTML/BrowsingContext.cpp | 18 ------------------ .../Libraries/LibWeb/HTML/BrowsingContext.h | 3 --- 2 files changed, 21 deletions(-) diff --git a/Userland/Libraries/LibWeb/HTML/BrowsingContext.cpp b/Userland/Libraries/LibWeb/HTML/BrowsingContext.cpp index 7e182c40ab..0ebe359f26 100644 --- a/Userland/Libraries/LibWeb/HTML/BrowsingContext.cpp +++ b/Userland/Libraries/LibWeb/HTML/BrowsingContext.cpp @@ -880,24 +880,6 @@ void BrowsingContext::discard() parent()->remove_child(*this); } -// https://html.spec.whatwg.org/multipage/window-object.html#close-a-browsing-context -void BrowsingContext::close() -{ - VERIFY(active_document()); - - // FIXME: 1. If the result of calling prompt to unload with browsingContext's active document is "refuse", then return. - - // 2. Unload browsingContext's active document. - active_document()->unload(); - - // 3. Remove browsingContext from the user interface (e.g., close or hide its tab in a tabbed browser). - if (m_page) - m_page->client().page_did_close_browsing_context(*this); - - // 4. Discard browsingContext. - discard(); -} - void BrowsingContext::append_child(JS::NonnullGCPtr child) { VERIFY(!child->m_parent); diff --git a/Userland/Libraries/LibWeb/HTML/BrowsingContext.h b/Userland/Libraries/LibWeb/HTML/BrowsingContext.h index 23f62e51d9..cdd2605da5 100644 --- a/Userland/Libraries/LibWeb/HTML/BrowsingContext.h +++ b/Userland/Libraries/LibWeb/HTML/BrowsingContext.h @@ -212,9 +212,6 @@ public: void discard(); bool has_been_discarded() const { return m_has_been_discarded; } - // https://html.spec.whatwg.org/multipage/window-object.html#close-a-browsing-context - void close(); - Optional const& creator_url() const { return m_creator_url; } virtual String const& window_handle() const override { return m_window_handle; }