286136 - Unnecessary to forward properties to textbox's HTML inputField

Status: NEW

(Core :: XUL, defect)

Description

[Aaron Leventhal]

Spunoff from from 286079.

In both versions of textbox.xml, we're forwarding property changes to the HTML
inputField, even though the input field inherits those attributes.

We should not need to do things like the second line of this snippet:
      <property name="readonly"
                onset="this.inputField.readonly = val;
                if (val) this.setAttribute('readonly', 'true');
                else this.removeAttribute('readonly'); return val;"


At least for disabled and tabIndex, I've tested and shown that it is unnecessary
extra code.

Comment 1

[Aaron Leventhal]

Attachment: Testcase of XUL textbox's tabIndex getting set dynamically

This testcase still works when the code |this.inputField.tabIndex = val;| is
removed from the tabIndex property setter.

Comment 2

[Boris Zbarsky [:bzbarsky]]

The only property we should really need to forward is 'value', I would think...