[epiphany/mcatanzaro/passwords: 1/15] ephy.js: Move forms code around



commit c64aeca34246c52b1050d7028cb394b3928dea59
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]