[wpt-sync] Sync PR 27510 - [shadow-dom] shadow-dom/nested-slot-remove-crash.html should use `let` instead of `const` to declare variables
Categories
(Core :: DOM: Core & HTML, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox87 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 27510 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/27510
Details from upstream follow.
b'Tetsuharu Ohzeki <tetsuharu.ohzeki@gmail.com>' wrote:
[shadow-dom] shadow-dom/nested-slot-remove-crash.html should use
let
instead ofconst
to declare variablesI cannot see https://crbug.com/1159328 by the permission problem, so I'm not sure about the original details. However, from the code comment, I doubt the current code does not works as intended.
- If this code runs on Chromium/Firefox/WebKit, this test does not crash.
- But by the browser's developer tools console, the execution is stopped at https://github.com/web-platform-tests/wpt/blob/0fa6dc568c006811e1fc0035d3b9c70dd7efadf5/shadow-dom/nested-slot-remove-crash.html#L30 by
SyntaxError
.
- By the comment, I seem this test is expected to reach https://github.com/web-platform-tests/wpt/blob/0fa6dc568c006811e1fc0035d3b9c70dd7efadf5/shadow-dom/nested-slot-remove-crash.html#L39-L40.
- By pull requests which imported this tests (https://github.com/web-platform-tests/wpt/pull/26951, https://github.com/web-platform-tests/wpt/pull/27001), I feel their original bug is related to DOM.
So I think this
root
should be declare bylet
instead ofconst
.Is this guess right?
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Comment 1•3 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=a3898d5962498aca7af3f437b110a938cd17406e
Assignee | ||
Comment 2•3 years ago
|
||
CI Results
Ran 14 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
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/231e9e2b8cdc [wpt PR 27510] - [shadow-dom] shadow-dom/nested-slot-remove-crash.html should use `let` instead of `const` to declare variables, a=testonly
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/eda94059722d [wpt PR 27510] - [shadow-dom] shadow-dom/nested-slot-remove-crash.html should use `let` instead of `const` to declare variables, a=testonly
Comment 5•3 years ago
|
||
bugherder |
Description
•