1755614 - Clean up editor/libeditor/tests/test_contenteditable_focus.html

Status: RESOLVED FIXED

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

Description

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

While I'm debugging my patch for bug 1755104, test_contenteditable_focus.html becomes orange, but it's hard to read which test fails because most tests are not chunked. So, it should be rewritten with IIFE and make each failure explain what the result should be and why the result should be.

Comment 1

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

Attachment: Bug 1755614 - Clean up test_contenteditable_focus.html r=m_kato!

It's not split to each chunk with IIFE nor the message explain what is the
expected result. This just rewrites it without changing what's are tested
except adding a call of testFocusMove with <button> in the main editor
(i.e., editable button).

Depends on D138876

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/80c3bbacd859
Clean up test_contenteditable_focus.html r=m_kato

Comment 3

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/80c3bbacd859