Bug 1612258 Comment 5 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

### QA verification notes

**Previous behavior**
The dismissed doorhanger would not show up when editing a password input field until the field was blurred; i.e. it only showed up on “change”[1] events.

**New behavior**
The dismissed doorhanger shows up when the field is edited, without requiring the field to be blurred; i.e. it shows on "input"[2] events.

**Unchanged behavior**
As before, we expect the values of the username and password field (when the password field is non-empty) in the update/save doorhanger to match the values in the form.

Autosave of the initial fill and subsequent edits of generated passwords should also work as before; i.e. editing a generated password field will get autosaved on "change" events.

_Known bugs that are out of scope_
* Bug 1612257 (i.e. the behavior of the doorhanger when an edit clears the password field after a dismissed doorhanger was previously shown).
  * This should just behave the same way it did before this patch.

[1]: https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/change_event
[2]: https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/input_event
### QA verification notes

**Previous behavior**
The dismissed doorhanger would not show up when editing a password input field until the field was blurred; i.e. it only showed up on "change"[1] events.

**New behavior**
The dismissed doorhanger shows up when the field is edited, without requiring the field to be blurred; i.e. it shows on "input"[2] events.

**Unchanged behavior**
As before, we expect the values of the username and password field (when the password field is non-empty) in the update/save doorhanger to match the values in the form.

Autosave of the initial fill and subsequent edits of generated passwords should also work as before; i.e. editing a generated password field will get autosaved on "change" events.

_Known bugs that are out of scope_
* Bug 1612257 (i.e. the behavior of the doorhanger when an edit clears the password field after a dismissed doorhanger was previously shown).
  * This should just behave the same way it did before this patch.

[1]: https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/change_event
[2]: https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/input_event

Back to Bug 1612258 Comment 5