1858977 - Access of window.screen in a detached contentWindow causes NS_ERROR_UNEXPECTED

Status: RESOLVED FIXED

(Core :: DOM: CSS Object Model, defect)

Description

[awathefox]

Steps to reproduce:

1. var elem = document.createElement('iframe')
2. document.body.appendChild(elem)
3. var win = elem.contentWindow
4. document.body.removeChild(elem)
5. win.screen
6. This happens

Actual results:

Uncaught NS_ERROR_UNEXPECTED exception with no further information.

Expected results:

An Screen object with all values equal to default value (0, false, etc.) (Edge, Chromium, Android System Webview)

Comment 1

[BugBot [:suhaib / :marco]]

The Bugbug bot thinks this bug should belong to the 'Core::DOM: Core & HTML' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Comment 2

[Hsin-Yi Tsai (she/her) [:hsinyi]]

https://drafts.csswg.org/cssom-view/#dom-window-screen says "The screen attribute must return the Screen object associated with the Window object" . Maybe it requires a bit clarity on the expected behavior when the screen isn't associated with a Window object?

Comment 3

[BugBot [:suhaib / :marco]]

The severity field is not set for this bug.
:jfkthame, could you have a look please?

For more information, please visit BugBot documentation.

Comment 4

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1858977 - Don't throw on screen getters. r=smaug,#webidl

The screen lifetime seems like it was a lot more complicated
in the past (looking at the comments in Window.webidl).

But nowadays it seems it can be more similar to the other
objects like VisualViewport etc.

Comment 5

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1858977 - Remove dead code to lock orientation on video controls. r=smaug,#webidl

mozLockOrientation etc haven't done anything for a while.

Depends on D193212

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bf2dfd2da634
Don't throw on screen getters. r=smaug,webidl
https://hg.mozilla.org/integration/autoland/rev/9565b6ba1cc8
Remove dead code to lock orientation on video controls. r=smaug,webidl,reusable-components-reviewers,mconley,hjones

Comment 7

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/43076 for changes under testing/web-platform/tests

Comment 8

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/bf2dfd2da634
https://hg.mozilla.org/mozilla-central/rev/9565b6ba1cc8

Comment 9

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot