782191 - Only begin streaming video/audio from camera/recording device when needed through mozGetUserMedia

Status: VERIFIED FIXED

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

Description

[Jason Smith [:jsmith]]

In comparing Chrome's production implementation to our Nightly implementation, one big difference I noticed was that they only appear to request access to the camera when they actually need it, not just immediately grab and lock the resource right after the mozGetUserMedia callback (i.e. lazy loading). This seems logical that they do this, as it prevents use of a resource that's not actively in use. We should follow suit here and only lock the camera for use when we actually need it post a mozGetUserMedia callback, and not do what we are doing now.

Comment 1

[Jason Smith [:jsmith]]

Confirmed this happens with the attached test case. We haven't started the content at all this test case - we've only made the gUM call and did console output. We shouldn't make the camera active in this case.

Comment 2

[Jason Smith [:jsmith]]

Attachment: Test Case

Comment 3

[Anant Narayanan [:anant]]

Bug 802411 should have fixed this.

Comment 4

[Jason Smith [:jsmith]]

Verified on 10/19 build. This needs a test too.

Comment 5

[Jason Smith [:jsmith]]

No longer valid behavior given changed requirements, so no need to track for in-testsuite.