Bug 1443318 - Set tabbox before tabContainer in gBrowser.init since the tabContainer's XBL constructor needs the tabbox. r=Gijs
authorDão Gottwald <dao@mozilla.com>
Thu, 08 Mar 2018 12:17:32 +0100
changeset 462195 3934ce46a591c36540aafef8b576723ad53e5951
parent 462194 def924646dff0c5ee32a82638037e611e283fe6a
child 462196 00c1789cdd6e6ae437589cd6e6c12f670f61d1a1
push id1683
push usersfraser@mozilla.com
push dateThu, 26 Apr 2018 16:43:40 +0000
treeherdermozilla-release@5af6cb21869d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersGijs
bugs1443318
milestone60.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1443318 - Set tabbox before tabContainer in gBrowser.init since the tabContainer's XBL constructor needs the tabbox. r=Gijs MozReview-Commit-ID: GH2HZk3gEyf
browser/base/content/tabbrowser.js
--- a/browser/base/content/tabbrowser.js
+++ b/browser/base/content/tabbrowser.js
@@ -23,22 +23,22 @@ window._gBrowser = {
 
     this.mPanelContainer = document.getElementById("tabbrowser-tabpanels");
     this.addEventListener = this.mPanelContainer.addEventListener.bind(this.mPanelContainer);
     this.removeEventListener = this.mPanelContainer.removeEventListener.bind(this.mPanelContainer);
     this.dispatchEvent = this.mPanelContainer.dispatchEvent.bind(this.mPanelContainer);
 
     this.initialBrowser = document.getElementById("tabbrowser-initialBrowser");
 
+    this.tabbox = document.getElementById("tabbrowser-tabbox");
+
     this.tabContainer = document.getElementById("tabbrowser-tabs");
 
     this.tabs = this.tabContainer.childNodes;
 
-    this.tabbox = document.getElementById("tabbrowser-tabbox");
-
     this.closingTabsEnum = { ALL: 0, OTHER: 1, TO_END: 2 };
 
     this._visibleTabs = null;
 
     this.mCurrentTab = null;
 
     this._lastRelatedTabMap = new WeakMap();