Closed
Bug 1335494
Opened 7 years ago
Closed 7 years ago
getUserMedia({audio:true, video:true}) succeeds but video frames (mozPaintedFrames) never advance
Categories
(Core :: WebRTC: Audio/Video, defect, P1)
Tracking
()
RESOLVED
DUPLICATE
of bug 1339816
People
(Reporter: jib, Assigned: achronop)
References
Details
(Keywords: intermittent-failure, regression)
Attachments
(1 file)
15.24 KB,
patch
|
Details | Diff | Splinter Review |
This is one of a couple of different "gUM doesn't respond bugs". A regression from bug 1315928. STR: 1. Open http://jsfiddle.net/jib1/f6tqv8oy/ 2. Hit the jsfiddle Run button and share cam+mic 3. Goto 2. Expected result: Live video and running numbers every time, e.g.: mozPaintedFrames: 267, currentTime: 9.198004535147392 Actual result: After a couple of tries: Frozen or no video, with mozPaintedFrames frozen at 0 while currentTime advances e.g.: mozPaintedFrames: 0, currentTime: 9.198004535147392 Regression range: 87:25.96 INFO: Last good revision: e820c6bc062ebdaf5c39cf257581a8c243371d85 87:25.96 INFO: First bad revision: bd9dc9379305055245e0751095b6e6bdeb214b32 87:25.96 INFO: Pushlog: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=e820c6bc062ebdaf5c39cf257581a8c243371d85&tochange=bd9dc9379305055245e0751095b6e6bdeb214b32
Reporter | ||
Updated•7 years ago
|
Keywords: intermittent-failure
Comment 1•7 years ago
|
||
Note: mac-only
Updated•7 years ago
|
Rank: 13
Reporter | ||
Comment 2•7 years ago
|
||
Monday's debugging in #media: http://logs.glob.uno/?c=mozilla%23media&s=30+Jan+2017&e=31+Jan+2017#c315903
Updated•7 years ago
|
Flags: needinfo?(achronop)
Assignee | ||
Comment 3•7 years ago
|
||
This is work in progress but when it is applied I am not able to repro the error. :padenot, :jib can you please try it?
Flags: needinfo?(padenot)
Flags: needinfo?(jib)
Reporter | ||
Updated•7 years ago
|
Assignee: nobody → achronop
Flags: needinfo?(jib)
Reporter | ||
Comment 4•7 years ago
|
||
Works!
Comment 5•7 years ago
|
||
In any case, I'd like to understand why this break or fixes it here. I've been investigating some, but I'll continue tomorrow.
Flags: needinfo?(padenot)
Comment 6•7 years ago
|
||
It's in the middle of beta cycle, in case there is any new issues after upgrading new lib version, you probably want to push the patch to nightly and beta for more tests.
Assignee | ||
Comment 7•7 years ago
|
||
Originally cubeb commit 581d4bc8b16e5f81fe299e2ab9897ec48da8610a intended to resolve the issue, however manual testing revealed some more errors. I am addressing those. Bug 1339816 created to import cubeb lib with the fix. I hold on for now expecting the new fixes to be landed. When everything is ready I will use this bug to import the cubeb library including the pending fixes.
Flags: needinfo?(achronop)
Assignee | ||
Comment 8•7 years ago
|
||
New fixes pushed for review in PR: https://github.com/kinetiknz/cubeb/pull/235
Updated•7 years ago
|
Assignee | ||
Updated•7 years ago
|
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
Comment 10•7 years ago
|
||
Fixed in 53 in the duplicate (cubeb update)
You need to log in
before you can comment on or make changes to this bug.
Description
•