1335494 - getUserMedia({audio:true, video:true}) succeeds but video frames (mozPaintedFrames) never advance

Status: RESOLVED DUPLICATE of bug 1339816

(Core :: WebRTC: Audio/Video, defect, P1)

Description

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

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

Comment 1

[Randell Jesup [:jesup] (needinfo me)]

Note: mac-only

Comment 2

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

Monday's debugging in #media: http://logs.glob.uno/?c=mozilla%23media&s=30+Jan+2017&e=31+Jan+2017#c315903

Comment 3

[Alex Chronopoulos [:achronop]]

Attachment: update-cubeb-upstream-06fa8a484

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?

Comment 4

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

Works!

Comment 5

[Paul Adenot (:padenot)]

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.

Comment 6

[Astley Chen (inactive)]

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.

Comment 7

[Alex Chronopoulos [:achronop]]

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.

Comment 8

[Alex Chronopoulos [:achronop]]

New fixes pushed for review in PR: https://github.com/kinetiknz/cubeb/pull/235

Comment 10

[Liz Henry (:lizzard) (relman/hg->git project)]

Fixed in 53 in the duplicate (cubeb update)