Closed Bug 1634249 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 23333 - Simplify script text modifications test

Categories

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

task

Tracking

()

RESOLVED FIXED
mozilla78
Tracking Status
firefox78 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Dominic Farolino <dom@chromium.org> wrote:

Simplify script text modifications test

This test was previously added in https://crrev.com/c/1091912/. The
test asserts the correct behavior, but does so in a way that relies
certain async script scheduling behavior which is not mandatory. This
CL changes the test to assert the same thing, but without relying on an
async script, and implicitly its scheduling.

Bug: 686281
Change-Id: I7b1ae20fb77cfefdfc68dd12a0db9be9e2314cfe
Reviewed-on: https://chromium-review.googlesource.com/2173476
WPT-Export-Revision: a09c3e95408d6a82c857630d5eefb5f211c86c7b

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core

CI Results

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

Total 68 tests

Status Summary

Firefox

OK : 1
PASS: 67
FAIL: 1

Chrome

OK : 1
PASS: 1

Safari

OK : 1
FAIL: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/html/semantics/scripting-1/the-script-element/script-text-modifications.html
Modify inline script element's text after prepare-a-script before evaluation: FAIL (Chrome: PASS, Safari: FAIL)

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/aa5aaef185f4 [wpt PR 23333] - Simplify script text modifications test, a=testonly
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/fbc3ad389e6b [wpt PR 23333] - Simplify script text modifications test, a=testonly
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
You need to log in before you can comment on or make changes to this bug.