1861905 - Gecko-only WPT failure: [["defaultparagraphseparator","p"],["formatblock","<pre>"]] "<p>[foo<p>bar]" compare innerHTML

Status: RESOLVED FIXED

(Core :: DOM: Editor, task, P3)

Description

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

["<p>[foo<br>bar]</p>",
    [["defaultparagraphseparator","p"],["formatblock","<pre>"]],
    "<pre>[foo<br>bar]</pre>",
    [true,true],
    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"formatblock":[false,false,"p",false,false,"pre"]}],

<pre>foo</pre><pre>bar</pre> should also be allowed for now like this test:

["<p>[foo<p>bar]",
    [["defaultparagraphseparator","div"],["formatblock","<pre>"]],
    [
      "<pre>[foo<br>bar]</pre>",
      "<pre>[foo</pre><pre>bar]</pre>",
    ],
    [true,true],
    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"formatblock":[false,false,"p",false,false,"pre"]}],

Comment 1

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

Attachment: Bug 1861905 - Make `[["defaultparagraphseparator","p"],["formatblock","<pre>"]] "<p>[foo<p>bar]"` allow to create 2 `<pre>` elements r=m_kato!

There is no consensus how to treat multiple lines at formatting block.
Therefore, in bug 1244437, I made expectations in such cases allow to use both
<br>s in new format block or create multiple format blocks for each line.
However, I forgot to change this case.

Depends on D192182

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cfb4369a270b
Make `[["defaultparagraphseparator","p"],["formatblock","<pre>"]] "<p>[foo<p>bar]"` allow to create 2 `<pre>` elements 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/42889 for changes under testing/web-platform/tests

Comment 4

[Noemi Erli[:noemi_erli]]

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

Comment 5

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

Upstream PR merged by moz-wptsync-bot