colin/browserpass-extension
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove call to requestAnimationFrame (#53)

Browse Source
This commit is contained in:
Maxim Baz
2019-03-31 00:35:28 +01:00
committed by GitHub
parent a4bee5cac7
commit 5c9c6d89ec
1 changed files with 19 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

40
src/inject.js
View File

@@ -132,31 +132,29 @@
// There is likely a field asking for OTP code, so do not submit form just yet
password_inputs[1].select();
} else {
window.requestAnimationFrame(function() {
// try to locate the submit button
var submit = find(SUBMIT_FIELDS, loginForm);
// try to locate the submit button
var submit = find(SUBMIT_FIELDS, loginForm);
// Try to submit the form, or focus on the submit button (based on user settings)
if (submit) {
if (request.autoSubmit) {
submit.click();
} else {
submit.focus();
}
// Try to submit the form, or focus on the submit button (based on user settings)
if (submit) {
if (request.autoSubmit) {
submit.click();
} else {
// There is no submit button.
// We need to keep focus somewhere within the form, so that Enter hopefully submits the form.
var password = find(PASSWORD_FIELDS, loginForm);
if (password) {
password.focus();
} else {
var username = find(USERNAME_FIELDS, loginForm);
if (username) {
username.focus();
}
submit.focus();
}
} else {
// There is no submit button.
// We need to keep focus somewhere within the form, so that Enter hopefully submits the form.
var password = find(PASSWORD_FIELDS, loginForm);
if (password) {
password.focus();
} else {
var username = find(USERNAME_FIELDS, loginForm);
if (username) {
username.focus();
}
}
});
}
}
// finished filling things successfully