1911592 - [wpt-sync] Sync PR 47458 - 32432 fixing panic when attempting loading scrdoc iframe

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, task, P4)

Description

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

Sync web-platform-tests PR 47458 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/47458
Details from upstream follow.

newmoneybigbucks <[e-mail]> wrote:

32432 fixing panic when attempting loading scrdoc iframe

<!-- Please describe your changes on the following line: -->
Changed get_referrer() function in globalscope.rs (in component/script/dom) so that substep 3.1.3 returns Referrer::NoReferrer if no browsing context, context parent, or parent document is found. This replaces calls to expect() and stops the program from panicking if none of those values are found.

Reviewed in servo/servo#32782

Comment 1

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

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=028167668ff262065ffd1ab8a24e00beca6359dc

Comment 2

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

CI Results

Ran 11 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests

Status Summary

Firefox

PASS: 1

Chrome

PASS: 1

Safari

PASS: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/eb42039f256e
[wpt PR 47458] - script: Properly handle removed iframes in `GlobalScope::get_referrer`, a=testonly

Comment 4

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/eb42039f256e