All users were logged out of Bugzilla on October 13th, 2018

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

VERIFIED FIXED

Status

()

VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: jsmith, Assigned: anant)

Tracking

Trunk
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [getUserMedia], [blocking-gum-])

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
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.
(Reporter)

Updated

6 years ago
Whiteboard: [getUserMedia]
(Reporter)

Updated

6 years ago
Keywords: qawanted
(Reporter)

Comment 1

6 years ago
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.
Keywords: qawanted
(Reporter)

Comment 2

6 years ago
Created attachment 652999 [details]
Test Case
(Reporter)

Updated

6 years ago
Assignee: nobody → mreavy
Whiteboard: [getUserMedia] → [getUserMedia], [blocking-gum-]
(Assignee)

Comment 3

6 years ago
Bug 802411 should have fixed this.
Assignee: mreavy → anant
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Reporter)

Updated

6 years ago
Keywords: verifyme
(Reporter)

Comment 4

6 years ago
Verified on 10/19 build. This needs a test too.
Status: RESOLVED → VERIFIED
Flags: in-testsuite?
Keywords: verifyme
(Reporter)

Comment 5

6 years ago
No longer valid behavior given changed requirements, so no need to track for in-testsuite.
Flags: in-testsuite? → in-testsuite-
You need to log in before you can comment on or make changes to this bug.