1701138 - layout/reftests/bugs/572598-ref.html should set `readonly` attribute to the `<input>` element

Status: RESOLVED FIXED

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

Description

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

Starting from the fix of bug 1220696, contentReadOnly command when an <input> or <textarea> has focus toggles readonly property of it instead of toggling the HTMLEditor's readonly flag. Except native theme of GTK, <input readonly> has same visual as <input>. Therefore, this bug has not been found when I work on bug 1220696.

Comment 1

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

Attachment: Bug 1701138 - Make 572598-ref.html set `readonly` attribute of the `<input>` element r=smaug!

Now, <input> element and <textarea> element handle contentReadOnly command
as toggling readonly attribute. Therefore, the result may be different if
and only if the theme changes the style with readonly attribute.

Therefore, the test result of 572598.html has been changed in logical, but
it's not been found because only GTK's native theme has different style for
<input> and <input readonly>.

This patch just changes the reference.

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1220696

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3c3d0c710ff8
Make 572598-ref.html set `readonly` attribute of the `<input>` element r=smaug

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/3c3d0c710ff8