Closed Bug 1304778 Opened 7 years ago Closed 7 years ago

Intermittent devtools/client/debugger/new/test/mochitest/browser_dbg-navigation.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. -


(DevTools :: Debugger, defect, P1)



(firefox50 unaffected, firefox51 unaffected, firefox52 fixed)

Firefox 52
Tracking Status
firefox50 --- unaffected
firefox51 --- unaffected
firefox52 --- fixed


(Reporter: intermittent-bug-filer, Assigned: bgrins)



(Keywords: intermittent-failure)


(1 file)

Priority: -- → P3
Let's go with a higher priority than that: one in a million, ASan or Win7 debug will hit this, but, on Linux32 debug it is actually permaorange.
Priority: P3 → --
Turns out permaorange was an exaggeration: I did a try run of something related, with 21 runs of this chunk, and only hit this on 20 of the 21.
Phil is right, this should be a P1.
Having said this, this should be a very simple one to fix:
- the test does pass
- it only takes a few seconds too much on Linux Debug (our slowest env)
--> So, simple fix here: increase the timeout for this test using requestLongerTimeout(2)

Eddy: can you make sure this bug gets handled by someone working on the debugger?
Flags: needinfo?(ejpbruel)
Priority: -- → P1
We could also skip this whole folder on linux 32 debug, as we do with the current debugger tests:
What do you think?  This might also allow us to proceed with Bug 1304178. Try push:
Comment on attachment 8798526 [details]
Bug 1304778 - Skip new debugger mochitests on linux 32 debug;
Attachment #8798526 - Flags: review?(jlong) → review+
Pushed by
Skip new debugger mochitests on linux 32 debug;r=jlongster
Assignee: nobody → bgrinstead
Blocks: 1304178
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
Flags: needinfo?(ejpbruel)
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.