Webrtc-pc: Processing Remote MediaStreamTracks should pass on receiver when constructing track event

Created on 20 Jul 2017  路  11Comments  路  Source: w3c/webrtc-pc

In the last step of 5.1.1 Processing Remote MediaStreamTracks:

Queue a task to fire an event named track with transceiver, track, and streams at the connection object.

It missed out the receiver attribute in RTCTrackEvent, which we know is simply getting from transceiver.receiver.

Test suite issue

Most helpful comment

Doh, I misread @stefhak's message as coming from @soareschen 馃檪

All 11 comments

I think this is on purpose, as you say the app can easily get hold of the receiver via the Transceiver.

pc.ontrack = e => e.tranceiver.receiver;

What I mean is the WebIDL definition for RTCTrackEvent has a receiver field. So that attribute should either be removed or filled with transceiver.receiver, no?

@soareschen Yeah that seems redundant to me. We should remove both the receiver readonly attribute and the corresponding member in RTCTrackEventInit IMHO. Good catch!

Since Firefox already supports e.receiver, we'd continue to for a while for legacy reasons.

@soareschen got it (https://github.com/w3c/webrtc-pc/issues/1499#issuecomment-319687209). Could you provide a PR?

PR up.

Thanks @jan-ivar!

@stefhak sorry I am rushing on another project, didn't get the time to work on it.

Doh, I misread @stefhak's message as coming from @soareschen 馃檪

I've closed https://github.com/w3c/webrtc-pc/pull/1532.

@soareschen can you add a PR to fill in event.receiver with transceiver.receiver instead?

Ping @soareschen !

Was this page helpful?
0 / 5 - 0 ratings

Related issues

jan-ivar picture jan-ivar  路  9Comments

alvestrand picture alvestrand  路  3Comments

soareschen picture soareschen  路  3Comments

ibsusu picture ibsusu  路  9Comments

aboba picture aboba  路  5Comments