[epiphany/mcatanzaro/passwords: 20/35] ephy.js: Move forms code around
- From: Jan-Michael Brummer <jbrummer src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [epiphany/mcatanzaro/passwords: 20/35] ephy.js: Move forms code around
- Date: Tue, 5 Mar 2019 20:53:42 +0000 (UTC)
commit 9260ec61c3cec502ca9a244a56fc1226f00afc23
Author: Michael Catanzaro <mcatanzaro igalia com>
Date: Sun Mar 3 15:20:50 2019 -0600
ephy.js: Move forms code around
It's weird that we have forms code at the top and bottom of the file,
followed by a bunch of unrelated stuff in the middle. Move it all to the
bottom.
Also, add and remove some semicolons.
embed/web-extension/resources/js/ephy.js | 132 +++++++++++++++----------------
1 file changed, 66 insertions(+), 66 deletions(-)
---
diff --git a/embed/web-extension/resources/js/ephy.js b/embed/web-extension/resources/js/ephy.js
index addd9b179..0e4205ddb 100644
--- a/embed/web-extension/resources/js/ephy.js
+++ b/embed/web-extension/resources/js/ephy.js
@@ -1,67 +1,5 @@
var Ephy = {};
-Ephy.formControlsAssociated = function(pageID, forms, serializer, rememberPasswords)
-{
- Ephy.formManagers = [];
-
- for (let i = 0; i < forms.length; i++) {
- if (!(forms[i] instanceof HTMLFormElement))
- continue;
- let formManager = new Ephy.FormManager(pageID, forms[i]);
- formManager.handleSensitiveElement(serializer);
- if (rememberPasswords)
- formManager.preFillForms();
- Ephy.formManagers.push(formManager);
- }
-}
-
-Ephy.handleFormSubmission = function(pageID, form, formAuthRequester)
-{
- let formManager = null;
- for (let i = 0; i < Ephy.formManagers.length; i++) {
- let manager = Ephy.formManagers[i];
- if (manager.pageID() == pageID && manager.form() == form) {
- formManager = manager;
- break;
- }
- }
-
- if (!formManager) {
- formManager = new Ephy.FormManager(pageID, form);
- Ephy.formManagers.push(formManager);
- }
-
- formManager.handleFormSubmission(formAuthRequester);
-}
-
-Ephy.hasModifiedForms = function()
-{
- for (let i = 0; i < document.forms.length; i++) {
- let form = document.forms[i];
- let modifiedInputElement = false;
- for (let j = 0; j < form.elements.length; j++) {
- let element = form.elements[j];
- if (!Ephy.isEdited(element))
- continue;
-
- if (element instanceof HTMLTextAreaElement)
- return !!element.value;
-
- if (element instanceof HTMLInputElement) {
- // A small heuristic here. If there's only one input element
- // modified and it does not have a lot of text the user is
- // likely not very interested in saving this work, so do
- // nothing (eg, google search input).
- if (element.value.length > 50)
- return true;
- if (modifiedInputElement)
- return true;
- modifiedInputElement = true;
- }
- }
- }
-}
-
Ephy.getWebAppTitle = function()
{
let metas = document.getElementsByTagName('meta');
@@ -307,7 +245,69 @@ Ephy.PreFillUserMenu = class PreFillUserMenu
if (menu)
menu.parentNode.removeChild(menu);
}
-};
+}
+
+Ephy.formControlsAssociated = function(pageID, forms, serializer, rememberPasswords)
+{
+ Ephy.formManagers = [];
+
+ for (let i = 0; i < forms.length; i++) {
+ if (!(forms[i] instanceof HTMLFormElement))
+ continue;
+ let formManager = new Ephy.FormManager(pageID, forms[i]);
+ formManager.handleSensitiveElement(serializer);
+ if (rememberPasswords)
+ formManager.preFillForms();
+ Ephy.formManagers.push(formManager);
+ }
+}
+
+Ephy.handleFormSubmission = function(pageID, form, formAuthRequester)
+{
+ let formManager = null;
+ for (let i = 0; i < Ephy.formManagers.length; i++) {
+ let manager = Ephy.formManagers[i];
+ if (manager.pageID() == pageID && manager.form() == form) {
+ formManager = manager;
+ break;
+ }
+ }
+
+ if (!formManager) {
+ formManager = new Ephy.FormManager(pageID, form);
+ Ephy.formManagers.push(formManager);
+ }
+
+ formManager.handleFormSubmission(formAuthRequester);
+}
+
+Ephy.hasModifiedForms = function()
+{
+ for (let i = 0; i < document.forms.length; i++) {
+ let form = document.forms[i];
+ let modifiedInputElement = false;
+ for (let j = 0; j < form.elements.length; j++) {
+ let element = form.elements[j];
+ if (!Ephy.isEdited(element))
+ continue;
+
+ if (element instanceof HTMLTextAreaElement)
+ return !!element.value;
+
+ if (element instanceof HTMLInputElement) {
+ // A small heuristic here. If there's only one input element
+ // modified and it does not have a lot of text the user is
+ // likely not very interested in saving this work, so do
+ // nothing (eg, google search input).
+ if (element.value.length > 50)
+ return true;
+ if (modifiedInputElement)
+ return true;
+ modifiedInputElement = true;
+ }
+ }
+ }
+}
Ephy.PasswordManager = class PasswordManager
{
@@ -328,7 +328,7 @@ Ephy.PasswordManager = class PasswordManager
_onQueryResponse(username, password, id)
{
- let element = this._takePendingPromise(id)
+ let element = this._takePendingPromise(id);
if (element) {
if (username !== '' && password !== '')
element.resolver({username, password});
@@ -367,7 +367,7 @@ Ephy.PasswordManager = class PasswordManager
_onQueryUsernamesResponse(users, id)
{
- let element = this._takePendingPromise(id)
+ let element = this._takePendingPromise(id);
if (element)
element.resolver(users);
}
@@ -685,4 +685,4 @@ Ephy.FormManager = class FormManager
return { 'usernameNode' : usernameNode, 'passwordNode' : passwordNode };
}
-};
+}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]