Closed Bug 1163263 Opened 9 years ago Closed 9 years ago

Intermittent browser_615394-SSWindowState_events.js | This test exceeded the timeout threshold.

Categories

(Firefox :: Session Restore, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 42
Tracking Status
firefox40 --- fixed
firefox41 --- fixed
firefox42 --- fixed
firefox-esr31 --- unaffected
firefox-esr38 --- unaffected

People

(Reporter: KWierso, Assigned: xidorn)

Details

(Keywords: intermittent-failure)

13:48:08 INFO - 1270 INFO TEST-PASS | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | [test_setBrowserState] window3 ready event count correct
13:48:08 INFO - 1271 INFO TEST-PASS | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | [test_setBrowserState] window971 busy event count correct
13:48:08 INFO - 1272 INFO TEST-PASS | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | [test_setBrowserState] window971 ready event count correct
13:48:08 INFO - 1273 INFO TEST-PASS | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | [test_setBrowserState] checked 2 windows
13:48:08 INFO - 1274 INFO prepping for test_undoCloseWindow
13:48:08 INFO - 1275 INFO running test_undoCloseWindow
13:48:08 INFO - 1276 INFO running
13:48:08 INFO - 1277 INFO TEST-PASS | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | undefined assertion name
13:48:08 INFO - 1278 INFO TEST-PASS | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | undefined assertion name
13:48:08 INFO - 1279 INFO running test_finish
13:48:08 INFO - 1280 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | This test exceeded the timeout threshold. It should be rewritten or split up. If that's not possible, use requestLongerTimeout(N), but only as a last resort. - expected PASS
13:48:08 INFO - MEMORY STAT vsize after test: 874577920
13:48:08 INFO - MEMORY STAT residentFast after test: 367501312
13:48:08 INFO - MEMORY STAT heapAllocated after test: 104368804
13:48:08 INFO - 1281 INFO TEST-OK | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | took 96097ms
13:48:08 INFO - [Child 26761] WARNING: nsWindow::GetNativeData not implemented for this type: file /builds/slave/m-in-lx-d-00000000000000000000/build/src/widget/PuppetWidget.cpp, line 1102
13:48:08 INFO - ++DOCSHELL 0xa5cc4c00 == 20 [pid = 26761] [id = 20]
13:48:08 INFO - ++DOMWINDOW == 60 (0xa5edec00) [pid = 26761] [serial = 61] [outer = (nil)]
With my patchset which removes the permission requirement of fullscreen, this becomes a permfail on e10s bc2 on Linux 32bit. We probably should increase the timeout factor further? (I don't understand why removing unrelated code can make this test run longer...)

My try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=91f278e449af

BTW, what's the default timeout for e10s bc tests? I guess 30s? It seems that this test sometimes runs for ~90s and now ~100s, probably we need a factor of 4. Agree with this change?
Flags: needinfo?(wkocher)
Worth a shot.
Flags: needinfo?(wkocher)
https://hg.mozilla.org/mozilla-central/rev/37f196c61129
Assignee: nobody → quanxunzhen
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
You need to log in before you can comment on or make changes to this bug.