567531 - browser_461634.js test failing on latest trunk

Status: RESOLVED FIXED

(SeaMonkey :: Session Restore, defect)

Description

[Misak Khachatryan]

Attachment: fixed test

mochitest-browser-chrome failed:
make[1]: *** [mochitest-browser-chrome] Error 1

Running chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js...
TEST-PASS | chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js | Only one browser window should be open initially
TEST-PASS | chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js | Closed tab list has the expected length
TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js | The correct amout of tabs are to be forgotten - Got 0, expected 2
TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js | Everything is set up. - Got 0, expected 2
TEST-PASS | chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js | Invalid window for forgetClosedTab throws
TEST-PASS | chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js | Invalid tab for forgetClosedTab throws
TEST-PASS | chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js | Invalid tab for forgetClosedTab throws
TEST-PASS | chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js | The correct amout of tabs was removed
TEST-PASS | chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js | All tabs specifically forgotten were indeed removed
TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js | ... and tabs not specifically forgotten weren't. - Got 0, expected 2
TEST-PASS | chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js | Only one browser window should be open eventually
TEST-INFO | checking window state
TEST-PASS | chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js | before wait for focus -- loaded: complete active window: <no window active> focused window: <no window focused> desired window: ([object ChromeWindow]) chrome://navigator/content/navigator.xul child window: ([object XPCNativeWrapper [object Window]]) data:text/html,<body%20onload="setTimeout(function%20()%20{%20document.getElementById('target').focus();%20},%2010);"><input%20id='target'></body> docshell visible: true
TEST-PASS | chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js | must wait for focus
TEST-PASS | chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js | waitForEvent called <type:focus, target[object XPCNativeWrapper [object HTMLDocument]]> -- loaded: complete active window: ([object ChromeWindow]) chrome://navigator/content/navigator.xul focused window: ([object XPCNativeWrapper [object Window]]) data:text/html,<body%20onload="setTimeout(function%20()%20{%20document.getElementById('target').focus();%20},%2010);"><input%20id='target'></body> desired window: ([object ChromeWindow]) chrome://navigator/content/navigator.xul child window: ([object XPCNativeWrapper [object Window]]) data:text/html,<body%20onload="setTimeout(function%20()%20{%20document.getElementById('target').focus();%20},%2010);"><input%20id='target'></body> docshell visible: true
TEST-PASS | chrome://mochikit/content/browser/suite/common/tests/browser/browser_461634.js | maybe run tests <load:true, focus:true> -- loaded: complete active window: ([object ChromeWindow]) chrome://navigator/content/navigator.xul focused window: ([object XPCNativeWrapper [object Window]]) data:text/html,<body%20onload="setTimeout(function%20()%20{%20document.getElementById('target').focus();%20},%2010);"><input%20id='target'></body> desired window: ([object ChromeWindow]) chrome://navigator/content/navigator.xul child window: ([object XPCNativeWrapper [object Window]]) data:text/html,<body%20onload="setTimeout(function%20()%20{%20document.getElementById('target').focus();%20},%2010);"><input%20id='target'></body> docshell visible: true


I did something weird in this test, because of lack of proper undoCloseTab implementation, but now mostly can use Firefox version.

Comment 1

[[e-mail]]

Comment on attachment 446881 [details] [diff] [review]
fixed test

>-     closedTabs = JSON.parse(ss.getClosedTabData(newWin));
>+    closedTabs = eval("(" + ss.getClosedTabData(newWin) + ")");
Why this change? (There may be others; a -w diff would help!)

Comment 2

[Misak Khachatryan]

Attachment: properly fixed test

Sorry, copypasted without thinking ...

Comment 3

[[e-mail]]

Comment on attachment 446952 [details] [diff] [review]
properly fixed test

>+    ss.setWindowState(newWin, JSON.stringify(test_state), true);
>+    
Nit: spaces on blank line.

This is a review by inspection only, I didn't try running the test.

Comment 4

[Misak Khachatryan]

Pushed: http://hg.mozilla.org/comm-central/rev/24fd84018f88