Bug 1287202 - Revert bug 1293513 to setup progress listeners for fields in a <form>. draft
authorMatthew Noorenberghe <mozilla@noorenberghe.ca>
Thu, 22 Jun 2017 21:58:55 -0700
changeset 599440 7c32e1f005a16ea145d4aabfe29e766cdb6e688c
parent 597021 bc9e028dbdc5c2f989175e6816dc864159133aee
child 599441 52f2f2799ffa7c55487203a913482a90aea0c958
push id65523
push usermozilla@noorenberghe.ca
push dateFri, 23 Jun 2017 05:03:08 +0000
bugs1287202, 1293513
milestone56.0a1
Bug 1287202 - Revert bug 1293513 to setup progress listeners for fields in a <form>. MozReview-Commit-ID: 32iv1M3hyBU
toolkit/components/passwordmgr/LoginManagerContent.jsm
--- a/toolkit/components/passwordmgr/LoginManagerContent.jsm
+++ b/toolkit/components/passwordmgr/LoginManagerContent.jsm
@@ -377,26 +377,24 @@ var LoginManagerContent = {
     this._fetchLoginsFromParentAndFillForm(formLike, window);
   },
 
   onDOMInputPasswordAdded(event, window) {
     if (!event.isTrusted) {
       return;
     }
 
+    this.setupProgressListener(window);
+
     let pwField = event.target;
     if (pwField.form) {
       // Fill is handled by onDOMFormHasPassword which is already throttled.
       return;
     }
 
-    // Only setup the listener for formless inputs.
-    // Capture within a <form> but without a submit event is bug 1287202.
-    this.setupProgressListener(window);
-
     let formLike = LoginFormFactory.createFromField(pwField);
     log("onDOMInputPasswordAdded:", pwField, formLike);
 
     let deferredTask = this._deferredPasswordAddedTasksByRootElement.get(formLike.rootElement);
     if (!deferredTask) {
       log("Creating a DeferredTask to call _fetchLoginsFromParentAndFillForm soon");
       this._formLikeByRootElement.set(formLike.rootElement, formLike);