Open Bug 1231131 Opened 7 years ago Updated 2 months ago

Turn CanvasCaptureMediaStream into CanvasCaptureMediaStreamTrack


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





(Reporter: pehrsons, Unassigned)


Per [1], we should move the `requestFrame()` method and `canvas` attribute to a MediaStreamTrack subclass and remove the CanvasCaptureMediaStream we have now.

Let's wait for the spec update before implementing.

I don't know how widely this is used, but a two-stage approach might be wise: add the new MediaStreamTrack subclass, then later remove the MediaStream subclass.
OK, here's the PR:

The changes are exactly like you would imagine.  A bunch of shuffling of properties are likely needed.

If you land this, then please don't forget to add [Deprecated="xxx"] to the old properties:  That might help us get rid of them faster.  I haven't looked into it, but WebIDL usage telemetry would be very helpful.  I think that's automatic, but it might pay to check.
backlog: --- → webrtc/webaudio+
Rank: 25
Priority: -- → P2
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3

Both Chrome and Safari implement CanvasCaptureMediaStreamTrack.
Only Firefox implementation CanvasCaptureMediaStream.

Flags: in-testsuite?
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.