Bug 453095 - gBrowser.browsers shouldn't try to look like a node list. r=gavin

browser/base/content/browser.js

@@ -4397,11 +4397,7 @@ nsBrowserAccess.prototype =
 
   isTabContentWindow : function(aWindow)
   {
-    var browsers = gBrowser.browsers;
-    for (var ctr = 0; ctr < browsers.length; ctr++)
-      if (browsers.item(ctr).contentWindow == aWindow)
-        return true;
-    return false;
+    return gBrowser.browsers.some(function (browser) browser.contentWindow == aWindow);
   }
 }
 

browser/base/content/tabbrowser.xml

@@ -1777,15 +1777,8 @@
       <property name="browsers" readonly="true">
        <getter>
           <![CDATA[
-            if (!this._browsers) {
-              var browsers = [];
-              var i;
-              browsers.item = function(i) {return this[i];}
-              for (i = 0; i < this.mTabContainer.childNodes.length; i++)
-                browsers.push(this.mTabContainer.childNodes[i].linkedBrowser);
-              this._browsers = browsers;
-            }
-            return this._browsers;
+            return this._browsers ||
+                   (this._browsers = Array.map(this.mTabs, function (tab) tab.linkedBrowser));
           ]]>
         </getter>
       </property>