Implement "script.realmDestroyed" event
Categories
(Remote Protocol :: WebDriver BiDi, task, P1)
Tracking
(firefox119 fixed)
| Tracking | Status | |
|---|---|---|
| firefox119 | --- | fixed |
People
(Reporter: whimboo, Assigned: Sasha)
References
()
Details
(Whiteboard: [webdriver:m8][wptsync upstream][webdriver:relnote])
Attachments
(2 files)
This event is needed for clients to handle situations when realms are getting destroyed / removed, eg. closing a frame or tab.
|
Reporter | |
Comment 1•2 years ago
|
||
So generally there is no command yet which allows to destroy a specific existent realm. That means realms will be only destroyed when the window object itself gets removed. Hereby I was first thinking about a possible event that we should register for, but we most likely can just rely on the destructor logic of the message handler framework.
That means when the module cache is destroyed it will call destroy() for each module. When the script module removes all the realms we should be able to fire individual events for each of the realms. My hope here is that the event can still be sent to the parent process.
|
|
Reporter | |
Updated•2 years ago
|
|
|
Reporter | |
Updated•1 year ago
|
|
Assignee | |
Updated•8 months ago
|
|
|
Assignee | |
Updated•8 months ago
|
|
|
Assignee | |
Comment 2•8 months ago
|
||
|
|
Assignee | |
Comment 3•8 months ago
|
||
Depends on D187550
Pushed by aborovova@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d966f217dd1c [bidi] Implement "script.realmDestroyed" event. r=webdriver-reviewers,jdescottes https://hg.mozilla.org/integration/autoland/rev/5b6b792e8a29 [wdspec] Add tests for "script.realmDestroyed" event. r=webdriver-reviewers,jdescottes
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/41938 for changes under testing/web-platform/tests
|
||
Comment 6•8 months ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/d966f217dd1c
https://hg.mozilla.org/mozilla-central/rev/5b6b792e8a29
Upstream PR merged by moz-wptsync-bot
|
|
Reporter | |
Updated•7 months ago
|
Description
•