Bug 493823 - One test from browser_394759 not being run, r=zeniko

browser/components/sessionstore/src/nsSessionStore.js

@@ -2766,7 +2766,7 @@ SessionStoreService.prototype = {
     let normalWindowIndex = 0;
     // try to find a non-popup window in this._closedWindows
     while (normalWindowIndex < this._closedWindows.length &&
-           this._closedWindows[normalWindowIndex].isPopup)
+           !!this._closedWindows[normalWindowIndex].isPopup)
       normalWindowIndex++;
     if (normalWindowIndex >= maxWindowsUndo)
       spliceTo = normalWindowIndex + 1;

browser/components/sessionstore/test/browser/browser_394759.js

@@ -113,7 +113,7 @@ function test() {
   
   function test_behavior (callback) {
     // helper function that does the actual testing
-    function openWindowRec(windowsToOpen, expectedResults) {
+    function openWindowRec(windowsToOpen, expectedResults, recCallback) {
       // do actual checking
       if (!windowsToOpen.length) {
         let closedWindowData = JSON.parse(ss.getClosedWindowData());
@@ -130,7 +130,7 @@ function test() {
            "There were " + oResults.normal + " normal windows to repoen");
 
         // cleanup & return
-        executeSoon(callback);
+        executeSoon(recCallback);
         return;
       }
       // hack to force window to be considered a popup (toolbar=no didn't work)
@@ -149,7 +149,7 @@ function test() {
           executeSoon(function() {
             window.close();
             executeSoon(function() {
-              openWindowRec(windowsToOpen, expectedResults);
+              openWindowRec(windowsToOpen, expectedResults, recCallback);
             });
           });
         }, true);