Bug 170845 - RTCPeerConnection#getStreamById(id)
Summary: RTCPeerConnection#getStreamById(id)
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-04-14 01:44 PDT by Philip Jägenstedt
Modified: 2017-06-14 02:33 PDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Philip Jägenstedt 2017-04-14 01:44:18 PDT
The getStreamById() method was removed together with many other MediaStream-related APIs in 2014:
https://github.com/w3c/webrtc-pc/pull/18

The stub implementation has been removed from Gecko:
https://bugzilla.mozilla.org/show_bug.cgi?id=1318163

It is being considered for removal in Chrome:
https://bugs.chromium.org/p/chromium/issues/detail?id=698163

If you believe that this is required for web compat, please reach on that bug or on blink-dev@chromium.org.
Comment 1 youenn fablet 2017-04-14 07:32:34 PDT
Agreed, we should probably remove getStreamByID, getRemoteStreams and getLocalStreams. Maybe removeStream as well although we should probably assess it at the same time as addStream/onaddstream.
Comment 2 Jon Lee 2017-06-13 19:00:01 PDT
Thoughts?
Comment 3 youenn fablet 2017-06-14 02:33:26 PDT
I think we should try to remove the whole legacy API.

Removing that one in particular is ok now that adapter.js has a shim for it (that might be removed in a different schedule as WebKit).