|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
59 bytes, text/x-review-board-request
|Details | Review|
The form autofill popup won't appear on an input which is auto-focused by the website.
It can be tested in https://luke-chang.github.io/autofill-demo/basic.html
The root cause of this bug is the same as bug 1311301 so I applied its approach in our "MarkAsAutofillField". It works except that the page is loaded in a new tab first time. That's because "AutofillProfileAutoCompleteSearch" hasn't finished registering when we call "MaybeStartControllingInput" in "MarkAsAutofillField". Therefore, "form-history" will be used alternatively. In this patch I changed the loader of "FormAutofillContent.jsm" back to "Cu.import" to initialize it in the beginning. I also removed the "_" prefix of "_identifyAutofillFields" to explicitly describe it as a public method. Hi Matt, Could you take a look? Thanks.
Comment on attachment 8840387 [details] Bug 1341582 - Apply form autofill to an already-focused input, https://reviewboard.mozilla.org/r/114906/#review116650 Thanks
Matt, Thanks for the review.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/0fac3630e21b Apply form autofill to an already-focused input, r=MattN