aboutsummaryrefslogtreecommitdiffstats
path: root/xul-ext/chrome
diff options
context:
space:
mode:
authorGuilhem Moulin <guilhem@fripost.org>2015-03-31 23:52:54 +0200
committerGuilhem Moulin <guilhem@fripost.org>2015-03-31 23:52:54 +0200
commitd98c3455f55d872ff2fd3983da981eb8c578d93b (patch)
treef013041049242af931028cf83a1e1233139d4d93 /xul-ext/chrome
parent25d8c5b95ca2277cef3e86b3d0cf18c08d9eadda (diff)
Consider "invisible" forms with visible fields.
A form can have a width/height of 0 even though its fields are visible in the browser. E.g., http://www.lenson.com document.forms[0].getBoundingClientRect(); { bottom: 196, height: 0, left: 37, right: 208, top: 196, width: 171, x: 37, y: 196 } document.forms[0][0].getBoundingClientRect(); { bottom: 448, height: 22, left: 60, right: 196, top: 426, width: 136, x: 60, y: 426 } document.forms[0][1].getBoundingClientRect(); { bottom: 495, height: 22, left: 60, right: 196, top: 473, width: 136, x: 60, y: 473 }
Diffstat (limited to 'xul-ext/chrome')
-rw-r--r--xul-ext/chrome/content/icevault.js8
1 files changed, 4 insertions, 4 deletions
diff --git a/xul-ext/chrome/content/icevault.js b/xul-ext/chrome/content/icevault.js
index 1eae3e1..d84c5b7 100644
--- a/xul-ext/chrome/content/icevault.js
+++ b/xul-ext/chrome/content/icevault.js
@@ -91,12 +91,12 @@ var icevault = (function() {
// refresh the form list
let forms = [], getForms = [];
for (let i = 0; i < state.document.forms.length; i++) {
- let form = state.document.forms[i], rect = form.getBoundingClientRect();
- if (rect.width == 0 || rect.height == 0)
- continue; // invisible form
+ let form = state.document.forms[i];
let fields = [], getFields = [];
for (let i = 0; i < form.elements.length; i++) {
- let field = form.elements[i];
+ let field = form.elements[i], rect = field.getBoundingClientRect();
+ if (rect.width == 0 || rect.height == 0)
+ continue; // invisible field
if (["text","email","password"].indexOf(field.type) > -1) {
// store only these kinds of fields
let getField = { name: field.name, type: field.type, value: field.value };