Closed Bug 1422234 Opened 7 years ago Closed 6 years ago

Pressing Enter after an anchor link causes the content in new line to associate with the link

Tracking

()

Status:

RESOLVED FIXED

Milestone:

mozilla60

Tracking Flags:

Tracking

Status

firefox-esr52

---

unaffected

firefox57

---

wontfix

firefox58

---

wontfix

firefox59

---

fixed

firefox60

---

fixed

People

(Reporter: bowang, Assigned: masayuki)

References

(Blocks 1 open bug)

Details

(Keywords: regression, testcase)

Attachments

(4 files)

ff_bug.html 7 years ago Bo Wang 112 bytes, text/html		Details
Bug 1422234 - part 1: HTMLEditRules::ReturnInParagraph() should adjust split point if caret position is positioned at edge of anchor element 6 years ago Masayuki Nakano [:masayuki] (he/him)(JST, +0900) 59 bytes, text/x-review-board-request	m_kato : review+ lizzard : approval-mozilla-beta+	Details
Bug 1422234 - part 2: Add wpt-tests to check if insertParagraph command splits or doesn't split inline elements at their edge 6 years ago Masayuki Nakano [:masayuki] (he/him)(JST, +0900) 59 bytes, text/x-review-board-request	m_kato : review+	Details
part 2 for Beta (modifying wpt manifest) 6 years ago Masayuki Nakano [:masayuki] (he/him)(JST, +0900) 29.47 KB, patch	masayuki : review+ lizzard : approval-mozilla-beta+	Details \| Diff \| Splinter Review

Bo Wang

Reporter

Description

•

7 years ago

Attached file ff_bug.html — Details

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/62.0.3202.94 Safari/537.36

Steps to reproduce:

Open the following html page, place cursor after the anchor link and press enter.
 
<html>
<div contenteditable="true"><div><a href="http://www.google.com">www.google.com</a></div></div>
</html>


Actual results:

In the new line, the content that input by user is associated with the link.


Expected results:

The new line content should not be associated with the link, as all the other browsers do.

Makoto Kato [:m_kato]

Updated

•

7 years ago

Priority: -- → P3

Makoto Kato [:m_kato]

Updated

•

7 years ago

Blocks: editor-blink-compat

YF (Yang)

Updated

•

7 years ago

Status: UNCONFIRMED → NEW

Has STR: --- → yes

status-firefox57: --- → affected

status-firefox58: --- → ?

status-firefox59: --- → affected

status-firefox-esr52: --- → unaffected

Ever confirmed: true

Keywords: testcase

YF (Yang)

Comment 1

•

7 years ago

Regression range:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=1986a6f181d7c97e9e90ea645ebc7397f1b2b322&tochange=7d39d921ab4241c3dc9dd9013ee5d3c950c084fa

Blocks: 1297414

Has Regression Range: --- → yes

Keywords: regression

Version: 57 Branch → 55 Branch

Andrew Overholt [:overholt]

Comment 2

•

6 years ago

Aryeh isn't working on Mozilla stuff (according to his bugzilla name) so hopefully Masayuki can look as he reviewed the changes in bug 1297414.

Flags: needinfo?(masayuki)

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

Assignee

Comment 3

•

6 years ago

Sure.

Tested briefly, then, I see interesting result:

test case 1: selection is set at end of text in <b>
https://jsfiddle.net/d_toybox/x0kohuo0/
Gecko:  <p><b>forcus this editor and type Enter</b></p><p><b><br></b></p>
Chrome: <p><b>forcus this editor and type Enter</b></p><p><b><br></b></p>

test case 2: selection is set at end of <b>
https://jsfiddle.net/d_toybox/x0kohuo0/1/
Gecko:  <p><b>forcus this editor and type Enter<br></b></p><p><br><b></b></p>
Chrome: <p><b>forcus this editor and type Enter</b></p><p><b><br></b></p>

(Only Gecko inserts <br> at the end of <b>)

test case 3: selection is set at end of <p>
https://jsfiddle.net/d_toybox/x0kohuo0/2/
Gecko:  <p><b>forcus this editor and type Enter</b></p><p><br></p>
Chrome: <p><b>forcus this editor and type Enter</b></p><p><b><br></b></p>

test case 4: selection is set at end of text in <a>
https://jsfiddle.net/d_toybox/x0kohuo0/3/
Gecko:  <p><a href="../">forcus this editor and type Enter</a></p><p><a href="../"><br></a></p>
Chrome: <p><a href="../">forcus this editor and type Enter</a></p><p><br></p>

(Only Gecko insert <a> to the new paragraph)

test case 5: selection is set at end of <a>
https://jsfiddle.net/d_toybox/x0kohuo0/4/
Gecko:  <p><a href="../">forcus this editor and type Enter<br></a></p><p><br><a href="../"></a></p>
Chrome: <p><a href="../">forcus this editor and type Enter</a></p><p><br></p>

(Only Gecko insert <a> to the new paragraph after <br>)

test case 6: selection is set at end of <p>
https://jsfiddle.net/d_toybox/x0kohuo0/5/
Gecko:  <p><a href="../">forcus this editor and type Enter</a></p><p><br></p>
Chrome: <p><a href="../">forcus this editor and type Enter</a></p><p><br></p>

Assignee: nobody → masayuki

Status: NEW → ASSIGNED

Flags: needinfo?(masayuki)

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

Assignee

Comment 4

•

6 years ago

Hmm, according to execCommand spec (unofficial draf), if cursor is at end of inline element, inline elements shouldn't be copied to the right paragraph. However, both Gecko and Blink does it (EdgeHTML's behavior is completely broken, it wraps both left and right paragraph with the inline elements). I think that actual behavior must make sense since in most cases, users keep typing text with same style even after insert new paragraph. So, I should file a spec bug for it.

On the other hand, it's difficult to say about <a> element behavior. Both older Gecko and Blink ignores only when caret is start or end of <a href="foo">. So, <a> and <a name="foo"> are separated same as other inline elements. I think that not copying <a href="foo"> makes sense.  However, I also think so for <a name="foo"> too.

Anyway, we should take traditional behavior for now.

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

Assignee

Comment 5

•

6 years ago

Filed spec bug here: https://github.com/w3c/editing/issues/172

Makoto Kato [:m_kato]

Updated

•

6 years ago

status-firefox57: affected → wontfix

status-firefox58: ? → wontfix

status-firefox59: affected → fix-optional

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

Assignee

Comment 6

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b68913bd31a891aa8b6134d31cc9c3b74c22d3d3

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

Assignee

Comment 7

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=6c1d4e8072fb7dc453a390486fb4d4c68a99dc91

Comment hidden (mozreview-request)

When Enter key is pressed at start or end of <a href="foo"> element, we
shouldn't split it (in other words, we shouldn't create empty <a href="foo">
element in new paragraph) because users must not want to keep editing *same*
link in new paragraph in most cases.

This patch adjusts HTMLEditRules::ReturnInParagraph() selection start point
locally when it gets selection.  If caret is at start of an <a href="foo">
element, moves caret to before the element.  If caret is at end of an
<a href="foo"> element, moves to after the element.

Review commit: https://reviewboard.mozilla.org/r/215204/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/215204/

Comment hidden (mozreview-request)

Spec bug: https://github.com/w3c/editing/issues/172

Inline elements in paragraph except anchor elements which have non-empty
href attribute should be created in new paragraph even when new ones are
empty because users must expect to keep writing some text with same style,
but for anchor elements with non-empty href attribute, users must not
expect to create same link in new paragraph in most cases.

Review commit: https://reviewboard.mozilla.org/r/215206/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/215206/

Code Review Bot [:reviewbot]

Comment 10

•

6 years ago

mozreview-review

Comment on attachment 8945014 [details]
Bug 1422234 - part 2: Add wpt-tests to check if insertParagraph command splits or doesn't split inline elements at their edge

https://reviewboard.mozilla.org/r/215206/#review220758


Static analysis found 918 defects in this patch (only the first 30 are reported here).
 - 918 defects found by mozlint

You can run this analysis locally with:
 - `./mach lint check path/to/file` (Python/Javascript/wpt)


If you see a problem in this automated review, please report it here: http://bit.ly/2y9N9Vx


::: testing/web-platform/tests/editing/data/insertparagraph.js:784
(Diff revision 1)
>      [["defaultparagraphseparator","p"],["insertparagraph",""]],
>      "<p><b>foo</b></p><p>{}bar</p>",
>      [true,true],
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:784
(Diff revision 1)
>      [["defaultparagraphseparator","p"],["insertparagraph",""]],
>      "<p><b>foo</b></p><p>{}bar</p>",
>      [true,true],
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:784
(Diff revision 1)
>      [["defaultparagraphseparator","p"],["insertparagraph",""]],
>      "<p><b>foo</b></p><p>{}bar</p>",
>      [true,true],
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:786
(Diff revision 1)
>      [true,true],
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 1)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'defaultparagraphseparator'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 1)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 1)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 1)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 1)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 1)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 1)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 1)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'insertparagraph'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 1)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 1)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 1)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 1)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 1)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:789
(Diff revision 1)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:789
(Diff revision 1)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:789
(Diff revision 1)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:791
(Diff revision 1)
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'defaultparagraphseparator'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'insertparagraph'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:794
(Diff revision 1)
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:794
(Diff revision 1)
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:794
(Diff revision 1)
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:796
(Diff revision 1)
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'defaultparagraphseparator'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'insertparagraph'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:799
(Diff revision 1)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:799
(Diff revision 1)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:799
(Diff revision 1)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:801
(Diff revision 1)
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'defaultparagraphseparator'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'insertparagraph'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:804
(Diff revision 1)
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:804
(Diff revision 1)
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:804
(Diff revision 1)
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:806
(Diff revision 1)
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'defaultparagraphseparator'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'insertparagraph'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:809
(Diff revision 1)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:809
(Diff revision 1)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:809
(Diff revision 1)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:811
(Diff revision 1)
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'defaultparagraphseparator'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'insertparagraph'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 1)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

Comment hidden (mozreview-request)

Comment on attachment 8945014 [details]
Bug 1422234 - part 2: Add wpt-tests to check if insertParagraph command splits or doesn't split inline elements at their edge

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/215206/diff/1-2/

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

Assignee

Comment 12

•

6 years ago

mozreview-review-reply

Comment on attachment 8945014 [details]
Bug 1422234 - part 2: Add wpt-tests to check if insertParagraph command splits or doesn't split inline elements at their edge

https://reviewboard.mozilla.org/r/215206/#review220758

These styles are for consistency with existing lines in the file.

Code Review Bot [:reviewbot]

Comment 13

•

6 years ago

mozreview-review

Comment on attachment 8945014 [details]
Bug 1422234 - part 2: Add wpt-tests to check if insertParagraph command splits or doesn't split inline elements at their edge

https://reviewboard.mozilla.org/r/215206/#review220762


Static analysis found 918 defects in this patch (only the first 30 are reported here).
 - 918 defects found by mozlint

You can run this analysis locally with:
 - `./mach lint check path/to/file` (Python/Javascript/wpt)


If you see a problem in this automated review, please report it here: http://bit.ly/2y9N9Vx


::: testing/web-platform/tests/editing/data/insertparagraph.js:784
(Diff revision 2)
>      [["defaultparagraphseparator","p"],["insertparagraph",""]],
>      "<p><b>foo</b></p><p>{}bar</p>",
>      [true,true],
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:784
(Diff revision 2)
>      [["defaultparagraphseparator","p"],["insertparagraph",""]],
>      "<p><b>foo</b></p><p>{}bar</p>",
>      [true,true],
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:784
(Diff revision 2)
>      [["defaultparagraphseparator","p"],["insertparagraph",""]],
>      "<p><b>foo</b></p><p>{}bar</p>",
>      [true,true],
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:786
(Diff revision 2)
>      [true,true],
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 2)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'defaultparagraphseparator'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 2)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 2)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 2)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 2)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 2)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 2)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 2)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'insertparagraph'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 2)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 2)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 2)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 2)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:787
(Diff revision 2)
>      {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b>foo[]</b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:789
(Diff revision 2)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:789
(Diff revision 2)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:789
(Diff revision 2)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b>foo</b></p><p><b>{}<br></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:791
(Diff revision 2)
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'defaultparagraphseparator'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'insertparagraph'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:792
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b>foo[]</b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:794
(Diff revision 2)
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:794
(Diff revision 2)
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:794
(Diff revision 2)
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b>foo</b></div><div><b>{}<br></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:796
(Diff revision 2)
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'defaultparagraphseparator'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'insertparagraph'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:797
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><b><i>foo[]</i></b></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:799
(Diff revision 2)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:799
(Diff revision 2)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:799
(Diff revision 2)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><b><i>foo</i></b></p><p><b><i>{}<br></i></b></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:801
(Diff revision 2)
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'defaultparagraphseparator'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'insertparagraph'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:802
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><b><i>foo[]</i></b></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:804
(Diff revision 2)
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:804
(Diff revision 2)
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:804
(Diff revision 2)
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><b><i>foo</i></b></div><div><b><i>{}<br></i></b></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:806
(Diff revision 2)
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'defaultparagraphseparator'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'insertparagraph'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:807
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<p><i><b>foo[]</b></i></p>",
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:809
(Diff revision 2)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:809
(Diff revision 2)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:809
(Diff revision 2)
> +    [["defaultparagraphseparator","div"],["insertparagraph",""]],
> +    "<p><i><b>foo</b></i></p><p><i><b>{}<br></b></i></p>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:811
(Diff revision 2)
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'defaultparagraphseparator'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: Missing space before value for key 'insertparagraph'. [eslint: key-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

::: testing/web-platform/tests/editing/data/insertparagraph.js:812
(Diff revision 2)
> +    {"defaultparagraphseparator":[false,false,"p",false,false,"div"],"insertparagraph":[false,false,"",false,false,""]}],
> +["<div><i><b>foo[]</b></i></div>",
> +    [["defaultparagraphseparator","p"],["insertparagraph",""]],
> +    "<div><i><b>foo</b></i></div><div><i><b>{}<br></b></i></div>",
> +    [true,true],
> +    {"defaultparagraphseparator":[false,false,"div",false,false,"p"],"insertparagraph":[false,false,"",false,false,""]}],

Error: A space is required after ','. [eslint: comma-spacing]

Makoto Kato [:m_kato]

Comment 14

•

6 years ago

mozreview-review

Comment on attachment 8945013 [details]
Bug 1422234 - part 1: HTMLEditRules::ReturnInParagraph() should adjust split point if caret position is positioned at edge of anchor element

https://reviewboard.mozilla.org/r/215204/#review220766

Attachment #8945013 - Flags: review?(m_kato) → review+

Makoto Kato [:m_kato]

Comment 15

•

6 years ago

mozreview-review

Comment on attachment 8945014 [details]
Bug 1422234 - part 2: Add wpt-tests to check if insertParagraph command splits or doesn't split inline elements at their edge

https://reviewboard.mozilla.org/r/215206/#review221116

Attachment #8945014 - Flags: review?(m_kato) → review+

Pulsebot

Comment 16

•

6 years ago

Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/6c912e6d282a
part 1: HTMLEditRules::ReturnInParagraph() should adjust split point if caret position is positioned at edge of anchor element r=m_kato
https://hg.mozilla.org/integration/autoland/rev/0637a8fc925b
part 2: Add wpt-tests to check if insertParagraph command splits or doesn't split inline elements at their edge r=m_kato

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

Assignee

Comment 17

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=101995a06341c2101ea326d3fc5d54272c3520ee

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

Assignee

Comment 18

•

6 years ago

Attached patch part 2 for Beta (modifying wpt manifest) — Details — Splinter Review

Attachment #8945365 - Flags: review+

Natalia Csoregi [:nataliaCs]

Comment 19

•

6 years ago

bugherder

https://hg.mozilla.org/mozilla-central/rev/6c912e6d282a
https://hg.mozilla.org/mozilla-central/rev/0637a8fc925b

Status: ASSIGNED → RESOLVED

Closed: 6 years ago

status-firefox60: --- → fixed

Resolution: --- → FIXED

Target Milestone: --- → mozilla60

Ryan VanderMeulen [:RyanVM]

Comment 20

•

6 years ago

Please request Beta approval on this when you get a chance.

Flags: needinfo?(masayuki)

Flags: in-testsuite+

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

Assignee

Comment 21

•

6 years ago

Comment on attachment 8945013 [details]
Bug 1422234 - part 1: HTMLEditRules::ReturnInParagraph() should adjust split point if caret position is positioned at edge of anchor element

Approval Request Comment
[Feature/Bug causing the regression]:
Regression of bug 1297414.

[User impact if declined]:
User keep editing link even after inserting new paragraph.

[Is this code covered by automated tests?]:
Yes, the following patch checks the expected behavior.

[Has the fix been verified in Nightly?]:
Yes.

[Needs manual test from QE? If yes, steps to reproduce]:
No, because of the new automated tests.

[List of other uplifts needed for the feature/fix]:
Only necessary the following patch for automated tests.

[Is the change risky?]:
No.

[Why is the change risky/not risky?]:
This only adjusts insertion point before splitting elements with Enter key only when caret is at edge of an anchor element with non-empty href attribute.

[String changes made/needed]:
No.

Flags: needinfo?(masayuki)

Attachment #8945013 - Flags: approval-mozilla-beta?

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

Assignee

Comment 22

•

6 years ago

Comment on attachment 8945365 [details] [diff] [review]
part 2 for Beta (modifying wpt manifest)

Approval Request Comment
This is automated tests for the expected behavior after landing the preceding patch. The different point from the patch landed on m-c is, only MANIFEST.json.

Attachment #8945365 - Flags: approval-mozilla-beta?

Liz Henry (:lizzard) (relman/hg->git project)

Comment 23

•

6 years ago

Comment on attachment 8945013 [details]
Bug 1422234 - part 1: HTMLEditRules::ReturnInParagraph() should adjust split point if caret position is positioned at edge of anchor element

I am happy to see this one fixed, let's bring it to 59 beta 7.
Thanks for the new tests!

Attachment #8945013 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Liz Henry (:lizzard) (relman/hg->git project)

Updated

•

6 years ago

Attachment #8945365 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Arthur Iakab [arthur_iakab]

Comment 24

•

6 years ago

Please update the patches. I encountered issues when imported the part 2 for beta.

Flags: needinfo?(masayuki)

Ryan VanderMeulen [:RyanVM]

Updated

•

6 years ago

Flags: needinfo?(masayuki)

Ryan VanderMeulen [:RyanVM]

Comment 25

•

6 years ago

bugherder uplift

https://hg.mozilla.org/releases/mozilla-beta/rev/1d6350b50913
https://hg.mozilla.org/releases/mozilla-beta/rev/0fe1f0b63fac

status-firefox59: fix-optional → fixed

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

Assignee

Comment 26

•

6 years ago

(In reply to Arthur Iakab [arthur_iakab] from comment #24)
> Please update the patches. I encountered issues when imported the part 2 for
> beta.

(In reply to Ryan VanderMeulen [:RyanVM] from comment #25)
> https://hg.mozilla.org/releases/mozilla-beta/rev/1d6350b50913
> https://hg.mozilla.org/releases/mozilla-beta/rev/0fe1f0b63fac

Oh, thank you!

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

Assignee

Updated

•

4 years ago

Blocks: 1357365

Thomas D. (:thomas8)

Updated

•

3 years ago

Summary: Press enter after an anchor link cause the content in new line associate with the link → Pressing Enter after an anchor link causes the content in new line to associate with the link

You need to log in before you can comment on or make changes to this bug.