Bug 1397365: Part 0 - Make the initialBrowser in tabbrowser.xml not load spurious about:blank. r?mconley draft
authorZibi Braniecki <zbraniecki@mozilla.com>
Wed, 13 Sep 2017 00:43:29 -0700
changeset 664674 13411e618c40811d8679f3318be9ae1533977088
parent 664635 9517eea4a1a5955618fc79d039f9f0282b7185ca
child 664675 7a09b2447e56a5a8e270136eeb7a14cdb39f3a92
push id79760
push userbmo:gandalf@aviary.pl
push dateThu, 14 Sep 2017 08:02:46 +0000
reviewersmconley
bugs1397365
milestone57.0a1
Bug 1397365: Part 0 - Make the initialBrowser in tabbrowser.xml not load spurious about:blank. r?mconley This patch makes it so that we don't load a useless about:blank viewer in the initial tab when creating a new browser window. MozReview-Commit-ID: BZEq414jwAL
browser/base/content/browser.js
browser/base/content/tabbrowser.xml
--- a/browser/base/content/browser.js
+++ b/browser/base/content/browser.js
@@ -1612,17 +1612,17 @@ var gBrowserInit = {
 
     let mm = window.messageManager;
     mm.addMessageListener("Browser:FirstPaint", function onFirstPaint() {
       mm.removeMessageListener("Browser:FirstPaint", onFirstPaint);
       firstBrowserPaintDeferred.resolve();
     });
 
     this._uriToLoadPromise.then(uriToLoad => {
-      if (!uriToLoad || uriToLoad == "about:blank") {
+      if (!uriToLoad) {
         return;
       }
 
       // We don't check if uriToLoad is a XULElement because this case has
       // already been handled before first paint, and the argument cleared.
       if (uriToLoad instanceof Ci.nsIArray) {
         let count = uriToLoad.length;
         let specs = [];
--- a/browser/base/content/tabbrowser.xml
+++ b/browser/base/content/tabbrowser.xml
@@ -23,16 +23,17 @@
                   onselect="if (event.target.localName == 'tabpanels') this.parentNode.updateCurrentBrowser();">
         <xul:tabpanels flex="1" class="plain" selectedIndex="0" anonid="panelcontainer">
           <xul:notificationbox flex="1" notificationside="top">
             <xul:hbox flex="1" class="browserSidebarContainer">
               <xul:vbox flex="1" class="browserContainer">
                 <xul:stack flex="1" class="browserStack" anonid="browserStack">
                   <xul:browser anonid="initialBrowser" type="content" message="true" messagemanagergroup="browsers"
                                primary="true"
+                               nodefaultsrc="true"
                                xbl:inherits="tooltip=contenttooltip,contextmenu=contentcontextmenu,autocompletepopup,selectmenulist,datetimepicker"/>
                 </xul:stack>
               </xul:vbox>
             </xul:hbox>
           </xul:notificationbox>
         </xul:tabpanels>
       </xul:tabbox>
       <children/>