Questionable failures in /mediacapture-record/MediaRecorder-peerconnection.https.html
Categories
(Core :: Audio/Video: Recording, defect, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox85 | --- | fixed |
People
(Reporter: jib, Assigned: jib, NeedInfo)
References
(Regression)
Details
(Keywords: regression)
Attachments
(8 files, 3 obsolete files)
|
47 bytes,
text/x-phabricator-request
|
Details | Review | |
|
47 bytes,
text/x-phabricator-request
|
Details | Review | |
|
47 bytes,
text/x-phabricator-request
|
Details | Review | |
|
47 bytes,
text/x-phabricator-request
|
Details | Review | |
|
47 bytes,
text/x-phabricator-request
|
Details | Review | |
|
Bug 1614055 - Fix bug in waitForReceivedCodec() helper to not wait forever + improve error handling.
47 bytes,
text/x-phabricator-request
|
Details | Review | |
|
47 bytes,
text/x-phabricator-request
|
Details | Review | |
|
47 bytes,
text/x-phabricator-request
|
Details | Review |
+++ This bug was initially created as a clone of Bug #1613976 +++
Unrelated to the timeouts in Bug 1613976, some of the failures here are questionable, so I've filed https://github.com/w3c/mediacapture-record/issues/194.
|
Assignee | |
Updated•5 years ago
|
|
|
Assignee | |
Comment 1•4 years ago
|
||
|
|
Assignee | |
Comment 2•4 years ago
|
||
Depends on D62281
|
|
Assignee | |
Comment 3•4 years ago
|
||
Depends on D62282
|
|
Assignee | |
Comment 4•4 years ago
|
||
Depends on D62283
|
|
Assignee | |
Updated•4 years ago
|
|
|
Assignee | |
Comment 5•4 years ago
|
||
|
|
Assignee | |
Comment 6•4 years ago
|
||
Depends on D62285
|
|
Assignee | |
Comment 7•4 years ago
|
||
Depends on D62328
|
|
Assignee | |
Comment 8•4 years ago
•
|
||
(In reply to Jan-Ivar Bruaroey [:jib] (needinfo? me) from comment #5)
Try wo/ini file https://treeherder.mozilla.org/#/jobs?repo=try&revision=e1c4dcc8c16666f6ed5579aad3e49bd122f0d14f
So MediaRecorder-no-sink.https.html—which I haven't modified, but is even simpler—times out after 30 seconds.
16:40:42 INFO - TEST-TIMEOUT | /mediacapture-record/MediaRecorder-no-sink.https.html | took 30789ms
16:40:42 INFO - TEST-START | /mediacapture-record/MediaRecorder-peerconnection.https.html
16:40:42 INFO - Closing window 46
16:43:57 INFO - Got timeout in harness
16:43:57 INFO - TEST-UNEXPECTED-TIMEOUT | /mediacapture-record/MediaRecorder-peerconnection.https.html | TestRunner hit external timeout (this may indicate a hang)
16:43:57 INFO - TEST-INFO took 195330ms
16:43:58 INFO - IOError on command, setting status to CRASH
I'll increase the timeout to "long" on that test as well, but I suspect it will join the other test here and be TEST-UNEXPECTED-TIMEOUT then.
While I found a bug in waitForReceivedCodec(), that particular helper was only used in a vp9 test we're already failing. All the tests we're timing out over look legit, so I worry our android emulators are simply too slow to test MediaRecorder + Camera & mic + peer connection all in one test x3. I can try splitting some of these tests up, but if that doesn't work I'd recommend disabling this test on android.
|
||
Comment 9•4 years ago
|
||
There are some r+ patches which didn't land and no activity in this bug for 2 weeks.
:jib, could you have a look please?
For more information, please visit auto_nag documentation.
|
|
Assignee | |
Comment 10•4 years ago
|
||
Will land these when things quiet down a bit.
|
|
Assignee | |
Updated•4 years ago
|
|
|
Assignee | |
Comment 11•4 years ago
|
||
Depends on D62281
|
|
Assignee | |
Comment 12•4 years ago
|
||
Depends on D62283
|
|
Assignee | |
Comment 13•4 years ago
|
||
Depends on D62281
|
||
Updated•4 years ago
|
|
|
||
Updated•4 years ago
|
|
|
||
Updated•4 years ago
|
|
|
Assignee | |
Comment 14•4 years ago
|
||
Depends on D62335
|
|
Assignee | |
Comment 15•4 years ago
|
||
Depends on D97228
|
|
||
Updated•4 years ago
|
|
||
Comment 16•4 years ago
|
||
Pushed by jbruaroey@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/fcdcb15f2f31 Remove unused name field in MediaRecorder-peerconnection.https.html test. r=bryce,bwc https://hg.mozilla.org/integration/autoland/rev/816a885a508d Avoid ambiguous mimeTypes in MediaRecorder-peerconnection.https.html for now. r=bryce,bwc https://hg.mozilla.org/integration/autoland/rev/bd33ea15c424 Avoid Promise constructor anti-pattern & reduce recording time to pass test on android. r=bryce,bwc https://hg.mozilla.org/integration/autoland/rev/2964bd8f65bb Run different tests on browsers that fail MediaRecorder.isTypeSupported(mimeType). r=bryce,bwc https://hg.mozilla.org/integration/autoland/rev/6776f5707aa9 Avoid Promise constructor anti-pattern & reduce recording time in MediaRecorder-no-sink.https.html as well. r=bryce,bwc https://hg.mozilla.org/integration/autoland/rev/a952853ed0ac Fix bug in waitForReceivedCodec() helper to not wait forever + improve error handling. r=bwc https://hg.mozilla.org/integration/autoland/rev/d452d0268379 Simplify MediaRecorder-peerconnection.https.html.ini r=pehrsons https://hg.mozilla.org/integration/autoland/rev/a9923b013e43 Optimize MediaRecorder-peerconnection.https.html test & make Chrome workarounds explicit r=pehrsons
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/26595 for changes under testing/web-platform/tests
|
||
Comment 18•4 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/fcdcb15f2f31
https://hg.mozilla.org/mozilla-central/rev/816a885a508d
https://hg.mozilla.org/mozilla-central/rev/bd33ea15c424
https://hg.mozilla.org/mozilla-central/rev/2964bd8f65bb
https://hg.mozilla.org/mozilla-central/rev/6776f5707aa9
https://hg.mozilla.org/mozilla-central/rev/a952853ed0ac
https://hg.mozilla.org/mozilla-central/rev/d452d0268379
https://hg.mozilla.org/mozilla-central/rev/a9923b013e43
Upstream PR merged by moz-wptsync-bot
|
||
Updated•3 years ago
|
|
||
Updated•3 years ago
|
Description
•