From 91efdd2a8912d7fa57b73b7b51a2b4e7f382cfb6 Mon Sep 17 00:00:00 2001
From: Jez Ng
Date: Sat, 28 Jan 2012 18:53:04 -0500
Subject: Make check for document.body more robust.

document.body.nodeName is not always 'body' -- it might be 'frameset',
for instance.
---
 vimiumFrontend.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

(limited to 'vimiumFrontend.js')

diff --git a/vimiumFrontend.js b/vimiumFrontend.js
index d3f802f5..954d192c 100644
--- a/vimiumFrontend.js
+++ b/vimiumFrontend.js
@@ -278,7 +278,7 @@ function scrollActivatedElementBy(x, y) {
       var lastElement = element;
       // we may have an orphaned element. if so, just scroll the body element.
       element = element.parentElement || document.body;
-    } while(lastElement.scrollTop == oldScrollTop && lastElement.nodeName.toLowerCase() != 'body');
+    } while(lastElement.scrollTop == oldScrollTop && lastElement != document.body);
   }
 
   if (x !== 0) {
@@ -288,7 +288,7 @@ function scrollActivatedElementBy(x, y) {
       element.scrollLeft += x;
       var lastElement = element;
       element = element.parentElement || document.body;
-    } while(lastElement.scrollLeft == oldScrollLeft && lastElement.nodeName.toLowerCase() != 'body');
+    } while(lastElement.scrollLeft == oldScrollLeft && lastElement != document.body);
   }
 
   // if the activated element has been scrolled completely offscreen, subsequent changes in its scroll
@@ -868,7 +868,7 @@ function performBackwardsFind() { findAndFocus(true); }
 
 function getLinkFromSelection() {
   var node = window.getSelection().anchorNode;
-  while (node && node.nodeName.toLowerCase() !== 'body') {
+  while (node && node !== document.body) {
     if (node.nodeName.toLowerCase() === 'a') return node;
     node = node.parentNode;
   }
-- 
cgit v1.2.3