Open Bug 1231131 Opened 7 years ago Updated 2 months ago

Turn CanvasCaptureMediaStream into CanvasCaptureMediaStreamTrack

Categories

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

defect

Tracking

()

People

(Reporter: pehrsons, Unassigned)

Details

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.


[1] https://github.com/w3c/mediacapture-fromelement/issues/16
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: https://github.com/w3c/mediacapture-fromelement/pull/18

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: https://developer.mozilla.org/en-US/docs/Mozilla/WebIDL_bindings#Deprecatedtag  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.