Closed Bug 1021776 Opened 6 years ago Closed 6 years ago

Intermittent test_getUserMedia_peerIdentity.html | video (protected) failed after waiting full duration

Categories

(Core :: WebRTC, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla33
Tracking Status
firefox31 --- unaffected
firefox32 --- fixed
firefox33 --- fixed
firefox-esr24 --- unaffected
b2g-v2.0 --- fixed
b2g-v2.1 --- fixed

People

(Reporter: emorley, Assigned: mt)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 3 obsolete files)

b2g_emulator_vm mozilla-central opt test mochitest-7 on 2014-06-05 19:57:38 PDT for push 90b7c043acc3

slave: tst-linux64-spot-1055

https://tbpl.mozilla.org/php/getParsedLog.php?id=41172038&tree=Mozilla-Central

{
20:04:18     INFO -  38 INFO TEST-START | /tests/dom/media/tests/mochitest/test_getUserMedia_peerIdentity.html
20:04:20     INFO -  TEST DEVICES: No test devices found (in media.{audio,video}_loopback_dev, using fake streams.
20:04:24     INFO -  *** UTM:SVC TimerManager:notify - notified @mozilla.org/b2g/webapps-update-timer;1
20:05:30     INFO -  39 INFO TEST-INFO | dumping last 12 message(s)
20:05:30     INFO -  40 INFO TEST-INFO | if you need more context, please use SimpleTest.requestCompleteLog() in your test
20:05:30     INFO -  41 INFO TEST-INFO | /tests/dom/media/tests/mochitest/test_getUserMedia_peerIdentity.html | getting media with constraints: {"audio":true,"video":true,"fake":true}
20:05:30     INFO -  42 INFO TEST-INFO | /tests/dom/media/tests/mochitest/test_getUserMedia_peerIdentity.html | got some audio samples: 1024
20:05:30     INFO -  43 INFO TEST-INFO | /tests/dom/media/tests/mochitest/test_getUserMedia_peerIdentity.html | without constraint: should not be silence for audio = OK
20:05:31     INFO -  44 INFO TEST-PASS | /tests/dom/media/tests/mochitest/test_getUserMedia_peerIdentity.html | audio is not silent
20:05:31     INFO -  45 INFO TEST-INFO | /tests/dom/media/tests/mochitest/test_getUserMedia_peerIdentity.html | waiting for video to start
20:05:31     INFO -  46 INFO TEST-INFO | /tests/dom/media/tests/mochitest/test_getUserMedia_peerIdentity.html | without constraint: should not throw on getImageData for video = OK
20:05:31     INFO -  47 INFO TEST-PASS | /tests/dom/media/tests/mochitest/test_getUserMedia_peerIdentity.html | video is not protected
20:05:31     INFO -  48 INFO TEST-INFO | /tests/dom/media/tests/mochitest/test_getUserMedia_peerIdentity.html | getting media with constraints: {"audio":true,"video":true,"fake":true,"peerIdentity":"user@example.com"}
20:05:31     INFO -  49 INFO TEST-INFO | /tests/dom/media/tests/mochitest/test_getUserMedia_peerIdentity.html | got some audio samples: 1024
20:05:31     INFO -  50 INFO TEST-INFO | /tests/dom/media/tests/mochitest/test_getUserMedia_peerIdentity.html | with constraint: should be silence for audio = OK
20:05:32     INFO -  51 INFO TEST-PASS | /tests/dom/media/tests/mochitest/test_getUserMedia_peerIdentity.html | audio is silent
20:05:32     INFO -  52 INFO TEST-INFO | /tests/dom/media/tests/mochitest/test_getUserMedia_peerIdentity.html | waiting for video to start
20:05:32     INFO -  53 INFO TEST-UNEXPECTED-FAIL | /tests/dom/media/tests/mochitest/test_getUserMedia_peerIdentity.html | video (protected) failed after waiting full duration
}
Flags: needinfo?(martin.thomson)
Acknowledged.  This is a performance-related issue that I've tested at least 100 times on my own machine, in a performance-limited VM, without failure.  But that's not all that satisfactory an answer, so I'll work something out.
Flags: needinfo?(martin.thomson)
(In reply to Martin Thomson [:mt] from comment #5)
> Acknowledged.  This is a performance-related issue that I've tested at least
> 100 times on my own machine, in a performance-limited VM, without failure. 
> But that's not all that satisfactory an answer, so I'll work something out.

Thank you :-)
Currently the #6 top orange on OrangeFactor :(
Yeah, it's been creeping up.  I'm spending my morning on it, which will either conclude with an answer, or a patch that disables the test.
(In reply to Martin Thomson [:mt] from comment #86)
> Yeah, it's been creeping up.  I'm spending my morning on it, which will
> either conclude with an answer, or a patch that disables the test.

That's great - thank you :-)
Comment on attachment 8441765 [details] [diff] [review]
0001-Bug-10021776-Disabling-peerIdentity-test-on-b2g.patch

I'll take any r=, bonus if you can check it in for me
Attachment #8441765 - Flags: review?(ryanvm)
Attachment #8441765 - Flags: review?(emorley)
Attachment #8441765 - Flags: review?(ryanvm)
Attachment #8441765 - Flags: review?(emorley)
Attachment #8441765 - Flags: review+
Keywords: checkin-needed
A few remarks:
- this looks like an ideal candidate for re-activating after our B2G emu tests run on real hardware
- I think checking for audio and video ever 200ms is most likely overloading the B2G emu - I would recommend to do this maybe every 3s or so
- I would also recommend to increase the over all timeouts from 20s to 60s
https://hg.mozilla.org/integration/mozilla-inbound/rev/f7874b813322
Assignee: nobody → martin.thomson
Flags: in-testsuite-
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/f7874b813322
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
Attachment #8446200 - Attachment is obsolete: true
Attachment #8446202 - Attachment is obsolete: true
Attachment #8446203 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.