Closed Bug 673285 Opened 10 years ago Closed 10 years ago
Investigate speeding up docshell/test/navigation/test
As part of the BuildFaster project (see https://wiki.mozilla.org/ReleaseEngineering/BuildFaster), we are investigating whether certain slow mochitests can be sped up. test_sessionhistory.html is a slow test, average execution time on a recent run (in ms): win32-debug, 46186 linux-opt, 28182 macosx64-debug, 51978 win32-opt, 28260 linux64-debug, 44187 linux64-opt, 28247 macosx64-opt, 28796 macosx-debug, 48526 linux-debug, 45130
Olli, as the original test author, would you be willing to help see if this test can be sped up?
Well, 7s or so is the 1000ms timeouts from nextTest(). I wonder why we need those.... Also some of the subtests have 1000ms timeouts. Past that, this is actually tests for 7 bugs in 1.... I wonder whether any particular subtests are slow here.
This patch removes the 1s timeouts. It passed on try. Do you think it's safe to land?
Attachment #553582 - Flags: review?(Olli.Pettay)
Attachment #553582 - Flags: review?(Olli.Pettay) → review+
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla9
You need to log in before you can comment on or make changes to this bug.