From a33baa347eda7b5f83ec99be29cf576e35b1f5c6 Mon Sep 17 00:00:00 2001 From: Ehsan Akhgari Date: Fri, 7 Dec 2012 08:27:39 -0500 Subject: [PATCH] Simplify browser_privatebrowsing_lastpbcontextexited.js and make it not leak, no bug DONTBUILD since this is NPOTB for global PB builds --HG-- extra : rebase_source : 6b2c4a18d04c09f707d6724f16b00c8f2e8ce1ff --- .../perwindow/browser_privatebrowsing_lastpbcontextexited.js | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/browser/components/privatebrowsing/test/browser/perwindow/browser_privatebrowsing_lastpbcontextexited.js b/browser/components/privatebrowsing/test/browser/perwindow/browser_privatebrowsing_lastpbcontextexited.js index c35ff876175d..f8ec7f580432 100644 --- a/browser/components/privatebrowsing/test/browser/perwindow/browser_privatebrowsing_lastpbcontextexited.js +++ b/browser/components/privatebrowsing/test/browser/perwindow/browser_privatebrowsing_lastpbcontextexited.js @@ -5,7 +5,7 @@ function test() { // We need to open a new window for this so that its docshell would get destroyed // when clearing the PB mode flag. - let newWin = window.openDialog(getBrowserURL(), "_blank", "chrome,all,dialog=no"); + let newWin = OpenBrowserWindow({private: true}); waitForExplicitFinish(); SimpleTest.waitForFocus(function() { let expected = false; @@ -14,12 +14,10 @@ function test() { is(aTopic, "last-pb-context-exited", "Correct topic should be dispatched"); is(expected, true, "notification not expected yet"); Services.obs.removeObserver(observer, "last-pb-context-exited", false); - gPrefService.clearUserPref("browser.privatebrowsing.keep_current_session"); finish(); } }; Services.obs.addObserver(observer, "last-pb-context-exited", false); - setPrivateWindow(newWin, true); expected = true; newWin.close(); // this will cause the docshells to leave PB mode newWin = null;