MediaEngineRemoteVideoSource Shutdown should not shut down Cameras()

RESOLVED FIXED in Firefox 47

Status

()

defect
P1
normal
Rank:
16
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: gcp, Assigned: jesup)

Tracking

Trunk
mozilla47
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox47 fixed, firefox49 fixed)

Details

Attachments

(1 attachment)

Reporter

Description

3 years ago
MediaEngineRemoteVideoSource::Shutdown currently calls the Shutdown function in CamerasChild. But we can have multiple active remote video sources, and shutting one down shouldn't tear down the entire subsystem. That can be done when MediaManager shuts down the respective backend.
gcp has a patch for this on that Try
Assignee: nobody → gpascutto
backlog: --- → webrtc/webaudio+
Rank: 16
Priority: -- → P1
Attachment #8722485 - Flags: review?(rjesup) → review+
Reporter

Comment 4

3 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/b1b50935ab45f360774e75a4d814b2f9bf02d916
Bug 1249313 - Don't shut down the entire Cameras IPC if a single source is shut down. r=jesup

Comment 5

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/b1b50935ab45
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
backed out for making bug 1207431 worse
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Reporter

Comment 8

3 years ago
I've given up on this for now.
Assignee: gpascutto → nobody
Assignee: nobody → rjesup
Depends on: 1267600

Comment 11

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/cf13730eeda2
Status: REOPENED → RESOLVED
Closed: 3 years ago3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.