Bug 1287642 - Open as dialog for Add-ons and Data Managers should no longer be associated with browser.link.open_external; r=Ratty, a=IanN
authorrsx11m <rsx11m.pub@gmail.com>
Tue, 09 Aug 2016 15:34:35 -0500
changeset 27455 0451d5cddfa9f01cf18c219f62872574aea482d2
parent 27454 ed85c9257236a59d81ded1ac7501c18f2d36077e
child 27456 30533da241d183964eafae47ed4a39967d5c253e
push id1850
push userclokep@gmail.com
push dateWed, 08 Mar 2017 19:29:12 +0000
treeherdercomm-esr52@028df196b2d9 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersRatty, IanN
bugs1287642
Bug 1287642 - Open as dialog for Add-ons and Data Managers should no longer be associated with browser.link.open_external; r=Ratty, a=IanN
suite/common/utilityOverlay.js
--- a/suite/common/utilityOverlay.js
+++ b/suite/common/utilityOverlay.js
@@ -1674,19 +1674,17 @@ function switchToTabHavingURI(aURI, aOpe
     if (browserWin.closed || browserWin == window)
       continue;
     if (switchIfURIInWindow(browserWin))
       return true;
   }
 
   // No opened tab has that url.
   if (aOpenNew) {
-    let newWindowPref = Services.prefs.getIntPref("browser.link.open_external");
-    let where = newWindowPref == kNewTab ? "tabfocused" : "window";
-    let browserWin = openUILinkIn(aURI.spec, where);
+    let browserWin = openUILinkIn(aURI.spec, "tabfocused");
     if (aCallback) {
       browserWin.addEventListener("pageshow", function browserWinPageShow(event) {
         if (event.target.location.href != aURI.spec)
           return;
         browserWin.removeEventListener("pageshow", browserWinPageShow, true);
         aCallback(browserWin.getBrowser().selectedBrowser);
       }, true);
     }