Bug 899104 Places should be able to open tabs in an active private window r=IanN a=Ratty SEAMONKEY_2_21b2_BUILD1 SEAMONKEY_2_21b2_RELEASE
authorNeil Rashbrook <neil@parkwaycc.co.uk>
Sun, 08 Sep 2013 20:50:58 +0100
changeset 15989 8b1a9ce8206ed2209dc0e876aba60e4c415adedb
parent 15988 0045d14fc33b9810f7c948d9461b9976577617dd
child 15990 709cd8192a5c74599e11ce7f898b51b94e6092c6
child 15992 2bb1ce0050b7d2c14cacb3a15cac158d64a71215
push id992
push userneil@parkwaycc.co.uk
push dateSun, 08 Sep 2013 19:53:30 +0000
treeherdercomm-beta@8b1a9ce8206e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersIanN, Ratty
bugs899104
Bug 899104 Places should be able to open tabs in an active private window r=IanN a=Ratty a=Callek for checkin to a CLOSED TREE
suite/common/src/PlacesUIUtils.jsm
--- a/suite/common/src/PlacesUIUtils.jsm
+++ b/suite/common/src/PlacesUIUtils.jsm
@@ -606,17 +606,18 @@ var PlacesUIUtils = {
   _showBookmarkDialog: function PUIU__showBookmarkDialog(aInfo) {
     var dialogURL = "chrome://communicator/content/bookmarks/bm-props.xul";
     var features = "centerscreen,chrome,modal,resizable=yes";
     this._getCurrentActiveWin().openDialog(dialogURL, "", features, aInfo);
     return ("performed" in aInfo && aInfo.performed);
   },
 
   _getTopBrowserWin: function PUIU__getTopBrowserWin() {
-    return Services.wm.getMostRecentWindow("navigator:browser");
+    return focusManager.activeWindow.gPrivate ||
+           Services.wm.getMostRecentWindow("navigator:browser");
   },
 
   _getCurrentActiveWin: function PUIU__getCurrentActiveWin() {
     return focusManager.activeWindow;
   },
 
   /**
    * Returns the closet ancestor places view for the given DOM node