1519957 - Remove textbox value saving

Status: RESOLVED FIXED

(Core :: XUL, enhancement, P3)

Description

[Neil Deakin]

Attachment: Remove textbox value caching

I assume the original intent of this was to save the value when the element is removed or changed, then reinserted. This relies on the box object, which could be deleted in some cases when the element is removed anyway, and only saves in the specific cases where the destructor is called.

Since it only works in some cases, for now at least, just remove this, as it is the only code that relies on the boxobject get/setProperty methods.

Comment 1

[:Paolo Amadini]

Comment on attachment 9036409 [details] [diff] [review]
Remove textbox value caching

Looks good to me, please wait until after the soft code freeze for landing.

I don't expect many regressions, but they'll be easy to fix if any is discovered.

Comment 2

[Neil Deakin]

Attachment: Part 1 - change textbox value caching to not use box object

So a change made in bug 1517480 now relies on the behaviour being removed by this patch, so I have added some simple support for it.

Comment 3

[Neil Deakin]

Attachment: Part 2 - set value property in login manager as well as attribute

Comment 4

[Matthew N. [:MattN]]

Comment on attachment 9046028 [details] [diff] [review]
Part 2 - set value property in login manager as well as attribute

Review of attachment 9046028 [details] [diff] [review]:
-----------------------------------------------------------------

::: toolkit/components/passwordmgr/LoginManagerPrompter.jsm
@@ +888,5 @@
>        chromeDoc.getElementById("password-notification-username")
>                 .setAttribute("placeholder", usernamePlaceholder);
> +      let nameField = chromeDoc.getElementById("password-notification-username");
> +      nameField.setAttribute("value", login.username);
> +      nameField.value = login.username;

Can you maybe update the comment above this? I guess once <textbox> is converted to CE this problem will go away.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/63908a1890a0
remove textbox value resetting that relying on box objects. This in some cases prevents the value from restoring but it doesn't restore in all useful cases anyway, r=dao
https://hg.mozilla.org/integration/mozilla-inbound/rev/57624be2e55c
fix login manager to not rely on the password field to not yet be initialized. This was broken by the other patch for bug 1519957, but using the .value property is what it should be doing anyway, but leave the attribute setting just in case, r=mattn

Comment 6

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/63908a1890a0
https://hg.mozilla.org/mozilla-central/rev/57624be2e55c