Merge branch 'master' of github.com:philc/vimium

1 files changed, 7 insertions, 7 deletions

diff --git a/linkHints.js b/linkHints.js
index 58d6e979..45f54a9d 100644
--- a/linkHints.js
+++ b/linkHints.js
@@ -71,7 +71,7 @@ function buildLinkHints() {
   // highlight all the links on the page; we don't want some link hints to have more chars than others.
   var digitsNeeded = Math.ceil(logXOfBase(visibleElements.length, settings.linkHintCharacters.length));
   var linkHintNumber = 0;
-  for (var i = 0; i < visibleElements.length; i++) {
+  for (var i = 0, count = visibleElements.length; i < count; i++) {
     hintMarkers.push(createMarkerFor(visibleElements[i], linkHintNumber, digitsNeeded));
     linkHintNumber++;
   }
@@ -104,7 +104,7 @@ function getVisibleClickableElements() {
   var visibleElements = [];
 
   // Find all visible clickable elements.
-  for (var i = 0; i < resultSet.snapshotLength; i++) {
+  for (var i = 0, count = resultSet.snapshotLength; i < count; i++) {
     var element = resultSet.snapshotItem(i);
     var clientRect = element.getClientRects()[0];
 
@@ -114,7 +114,7 @@ function getVisibleClickableElements() {
     // If the link has zero dimensions, it may be wrapping visible
     // but floated elements. Check for this.
     if (clientRect && (clientRect.width == 0 || clientRect.height == 0)) {
-      for (var j = 0; j < element.children.length; j++) {
+      for (var j = 0, childrenCount = element.children.length; j < childrenCount; j++) {
         if (window.getComputedStyle(element.children[j], null).getPropertyValue('float') != 'none') {
           var childClientRect = element.children[j].getClientRects()[0];
           if (isVisible(element.children[j], childClientRect)) {
@@ -248,8 +248,9 @@ function highlightLinkMatches(searchString) {
     if (linkMarker.getAttribute("hintString").indexOf(searchString) == 0) {
       if (linkMarker.style.display == "none")
         linkMarker.style.display = "";
-      for (var j = 0; j < linkMarker.childNodes.length; j++)
-        linkMarker.childNodes[j].className = (j >= searchString.length) ? "" : "matchingCharacter";
+      var childNodes = linkMarker.childNodes;
+      for (var j = 0, childNodesCount = childNodes.length; j < childNodesCount; j++)
+        childNodes[j].className = (j >= searchString.length) ? "" : "matchingCharacter";
       linksMatched.push(linkMarker.clickableItem);
     } else {
       linkMarker.style.display = "none";
@@ -274,8 +275,7 @@ function numberToHintString(number, numHintDigits) {
   } while (number > 0);
 
   // Pad the hint string we're returning so that it matches numHintDigits.
-  var hintStringLength = hintString.length;
-  for (var i = 0; i < numHintDigits - hintStringLength; i++)
+  for (var i = 0; i < numHintDigits - hintString.length; i++)
     hintString.unshift(settings.linkHintCharacters[0]);
   return hintString.join("");
 }