Closed
Bug 1112967
Opened 9 years ago
Closed 9 years ago
Consolidate the requestFlakyTimeout() calls
Categories
(Core :: WebRTC, defect)
Tracking
()
RESOLVED
FIXED
mozilla37
People
(Reporter: drno, Assigned: drno)
Details
Attachments
(1 file)
52.02 KB,
patch
|
bwc
:
review+
|
Details | Diff | Splinter Review |
As of 12-12 we have 54 calls to SimpleTest.requestFlakyTimeout() in our mochitests. This is ridiculous. And any new test which uses the queues from the templates.js fails with a bizarre error message.
Assignee | ||
Comment 1•9 years ago
|
||
Try run: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=da36bb0cf503
Assignee | ||
Updated•9 years ago
|
Attachment #8538244 -
Flags: review?(docfaraday)
Comment 2•9 years ago
|
||
Comment on attachment 8538244 [details] [diff] [review] consolidate_requestFklayTimeout.patch Review of attachment 8538244 [details] [diff] [review]: ----------------------------------------------------------------- lgtm
Attachment #8538244 -
Flags: review?(docfaraday) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Comment 3•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/99d65d6cb0b1
Keywords: checkin-needed
Comment 4•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/99d65d6cb0b1
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
You need to log in
before you can comment on or make changes to this bug.
Description
•