Bug 1135451 - fillForm() cleanup part E: remove didFillForm. r=MattN

This commit is contained in:
Justin Dolske 2015-02-27 15:47:37 -08:00
parent 41459f1c9f
commit 2fbcaf9cd5

View File

@ -742,7 +742,6 @@ var LoginManagerContent = {
// We will always have a selectedLogin at this point.
var didFillForm = false;
if (autofillForm && !isFormDisabled) {
// Fill the form
@ -764,18 +763,17 @@ var LoginManagerContent = {
if (passwordField.value != selectedLogin.password) {
passwordField.setUserInput(selectedLogin.password);
}
didFillForm = true;
} else if (!autofillForm) {
log("autofillForms=false but form can be filled; notified observers");
recordAutofillResult(AUTOFILL_RESULT.NO_AUTOFILL_FORMS);
return;
} else if (isFormDisabled) {
log("autocomplete=off but form can be filled; notified observers");
recordAutofillResult(AUTOFILL_RESULT.AUTOCOMPLETE_OFF);
return;
}
if (didFillForm) {
recordAutofillResult(AUTOFILL_RESULT.FILLED);
}
recordAutofillResult(AUTOFILL_RESULT.FILLED);
},
};