Bug 345075: Fix loadOneTab to ensure that the parent isn't set when passing null as the loadInBackground argument, r=mconnor

git-svn-id: svn://10.0.0.236/trunk@232757 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
gavin%gavinsharp.com 2007-08-22 05:04:23 +00:00
parent ebdd27fbd1
commit 8fcd966530

View File

@ -1019,11 +1019,11 @@
<parameter name="aAllowThirdPartyFixup"/>
<body>
<![CDATA[
var owner = aLoadInBackground ? null : this.selectedTab;
var tab = this.addTab(aURI, aReferrerURI, aCharset, aPostData, owner,
aAllowThirdPartyFixup);
var bgLoad = (aLoadInBackground != null) ? aLoadInBackground :
this.mPrefs.getBoolPref("browser.tabs.loadInBackground");
var owner = bgLoad ? null : this.selectedTab;
var tab = this.addTab(aURI, aReferrerURI, aCharset, aPostData, owner,
aAllowThirdPartyFixup);
// Set newly selected tab after quick timeout, otherwise hideous focus problems
// can occur when "browser.tabs.loadInBackground" is false and presshell is not ready
if (!bgLoad) {