Clean up MediaStream handling in GetUserMedia

RESOLVED FIXED in mozilla19

Status

()

defect
P2
normal
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: jesup, Assigned: jesup)

Tracking

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

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [getUserMedia] [blocking-gum+] [qa-])

Attachments

(1 attachment)

See the patch:

a) Audio was never calling EndTrack()
b) stream->Finish() wasn't explicitly called at the MediaManager level; it was sitting in the Video track handling and so wouldn't happen for an audio-only capture
c) mState transitions need to occur in the correct order versus stream calls (i.e. AddTrack, EndTrack, etc)
d) Audio should have the same protection against trying to push data into an ended track as video
Attachment #672351 - Flags: review?(roc)
Priority: -- → P2
Whiteboard: [getUserMedia] [blocking-gum+]
Comment on attachment 672351 [details] [diff] [review]
Clean up MediaStream interactions in gUM

Review of attachment 672351 [details] [diff] [review]:
-----------------------------------------------------------------

Sorry I didn't catch these for you.
Attachment #672351 - Flags: review?(roc) → review+
https://hg.mozilla.org/mozilla-central/rev/f7cb509ab5b0
Assignee: nobody → rjesup
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
Whiteboard: [getUserMedia] [blocking-gum+] → [getUserMedia] [blocking-gum+] [qa-]
You need to log in before you can comment on or make changes to this bug.