mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2025-07-29 20:29:18 +00:00
LibWeb/HTML: Inform navigation API about frame container destruction
This commit is contained in:
parent
022cd1adca
commit
6032827fe7
Notes:
github-actions[bot]
2025-04-27 11:55:11 +00:00
Author: https://github.com/shannonbooth
Commit: 6032827fe7
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/4487
Reviewed-by: https://github.com/trflynn89
3 changed files with 17 additions and 1 deletions
|
@ -2345,6 +2345,19 @@ bool Navigable::has_a_rendering_opportunity() const
|
|||
return browsing_context->page().client().is_ready_to_paint();
|
||||
}
|
||||
|
||||
// https://html.spec.whatwg.org/multipage/nav-history-apis.html#inform-the-navigation-api-about-child-navigable-destruction
|
||||
void Navigable::inform_the_navigation_api_about_child_navigable_destruction()
|
||||
{
|
||||
// 1. Inform the navigation API about aborting navigation in navigable.
|
||||
inform_the_navigation_api_about_aborting_navigation();
|
||||
|
||||
// FIXME: 2. Let navigation be navigable's active window's navigation API.
|
||||
|
||||
// FIXME: 3. Let traversalAPIMethodTrackers be a clone of navigation's upcoming traverse API method trackers.
|
||||
|
||||
// FIXME: 4. For each apiMethodTracker of traversalAPIMethodTrackers: reject the finished promise for apiMethodTracker with a new "AbortError" DOMException created in navigation's relevant realm.
|
||||
}
|
||||
|
||||
// https://html.spec.whatwg.org/multipage/nav-history-apis.html#inform-the-navigation-api-about-aborting-navigation
|
||||
void Navigable::inform_the_navigation_api_about_aborting_navigation()
|
||||
{
|
||||
|
|
|
@ -186,6 +186,8 @@ public:
|
|||
bool has_session_history_entry_and_ready_for_navigation() const { return m_has_session_history_entry_and_ready_for_navigation; }
|
||||
void set_has_session_history_entry_and_ready_for_navigation();
|
||||
|
||||
void inform_the_navigation_api_about_child_navigable_destruction();
|
||||
|
||||
bool has_pending_navigations() const { return !m_pending_navigations.is_empty(); }
|
||||
|
||||
template<typename T>
|
||||
|
|
|
@ -285,7 +285,8 @@ void NavigableContainer::destroy_the_child_navigable()
|
|||
return;
|
||||
navigable->set_has_been_destroyed();
|
||||
|
||||
// FIXME: 4. Inform the navigation API about child navigable destruction given navigable.
|
||||
// 4. Inform the navigation API about child navigable destruction given navigable.
|
||||
navigable->inform_the_navigation_api_about_child_navigable_destruction();
|
||||
|
||||
// 5. Destroy a document and its descendants given navigable's active document.
|
||||
navigable->active_document()->destroy_a_document_and_its_descendants(GC::create_function(heap(), [this, navigable] {
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue