Closed
Bug 825510
Opened 11 years ago
Closed 11 years ago
Pref on peer connection mochitest smoke test automation by default
Categories
(Core :: WebRTC, defect, P2)
Core
WebRTC
Tracking
()
RESOLVED
FIXED
mozilla21
People
(Reporter: jsmith, Assigned: jesup)
References
(Depends on 1 open bug)
Details
(Whiteboard: [WebRTC][blocking-webrtc+][qa-])
Attachments
(2 files)
1.27 KB,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
3.88 KB,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
When the issues blocking the ability to turn this on by default in mozilla central are fixed, we'll need to turn on this automation by default. This bug tracks that work.
Reporter | ||
Updated•11 years ago
|
Whiteboard: [WebRTC][blocking-webrtc+]
Reporter | ||
Updated•11 years ago
|
Updated•11 years ago
|
Assignee: nobody → hskupin
Priority: -- → P2
Updated•11 years ago
|
Assignee | ||
Comment 1•11 years ago
|
||
I left MOZ_WEBRTC_LEAKING_TESTS because we may need to disable individual tests temporarily
Assignee | ||
Updated•11 years ago
|
Attachment #704879 -
Flags: review?(ted)
Comment 2•11 years ago
|
||
Comment on attachment 704879 [details] [diff] [review] enable PeerConnection mochitests Review of attachment 704879 [details] [diff] [review]: ----------------------------------------------------------------- ::: dom/media/tests/mochitest/Makefile.in @@ +25,5 @@ > $(NULL) > > # The following tests are leaking and cannot be run by default yet > ifdef MOZ_WEBRTC_LEAKING_TESTS > MOCHITEST_FILES += \ Is there any reason to leave this block here?
Attachment #704879 -
Flags: review?(ted) → review+
Assignee | ||
Comment 3•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/528eac92e500 > Is there any reason to leave this block here? See comment 1
Target Milestone: --- → mozilla21
Comment 4•11 years ago
|
||
We are ok in doing that already? I can remember a leak we have seen on inbound because of that I haven't continued on this bug yet.
Assignee: hskupin → rjesup
Assignee | ||
Comment 5•11 years ago
|
||
runTest() now takes a desktopsupportedonly flag that wasn't set in the PeerConnection tests
Assignee | ||
Updated•11 years ago
|
Attachment #705030 -
Flags: review?(ted)
Assignee | ||
Comment 6•11 years ago
|
||
runTest() is in dom/media/tests/mochitests/head.js, FYI
Updated•11 years ago
|
Attachment #705030 -
Flags: review?(ted) → review+
Assignee | ||
Comment 7•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/4d567ebd2624
Comment 8•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/528eac92e500 https://hg.mozilla.org/mozilla-central/rev/4d567ebd2624
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Reporter | ||
Updated•11 years ago
|
Whiteboard: [WebRTC][blocking-webrtc+] → [WebRTC][blocking-webrtc+][qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•