1922712 - The new tests added in bug 1920646 fail only on the CI for wpt.fyi

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect)

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

I'm still not sure what's going on. ERROR message: SyntaxError: An invalid or illegal string was specified appears, but the line number is not available.

Comment 1

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Bug 1922712 - Make `editing/plaintext-only` stop doing `.contentEditable = "plaintext-only"` r=m_kato!,#dom-core

If dom.element.contenteditable.plaintext-only.enabled is false,
the API throws an exception. Therefore, the following tests cannot check
the result. So, we should not use it. Instead,
.setAttribute("contenteditable", "plaintext-only") is fine.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d7c82075e3ef
Make `editing/plaintext-only` stop doing `.contentEditable = "plaintext-only"` r=m_kato

Comment 3

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

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

Comment 4

[Serban Stanca [:SerbanS]]

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

Comment 5

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

Upstream PR merged by moz-wptsync-bot

Comment 6

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Now, the tests run all in their CI too.