Andreas Pehrson [:pehrsons] Assignee
	Description • 9 years ago

This seems trivial but wasn't covered by any other bug. We should implement this to make the API complete.

	Andreas Pehrson [:pehrsons] Assignee
	Comment 1 • 8 years ago

Review commit: https://reviewboard.mozilla.org/r/54244/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/54244/

	Andreas Pehrson [:pehrsons] Assignee
	Comment 2 • 8 years ago

Review commit: https://reviewboard.mozilla.org/r/54246/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/54246/

	Andreas Pehrson [:pehrsons] Assignee
	Comment 3 • 8 years ago

Review commit: https://reviewboard.mozilla.org/r/54248/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/54248/

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 4 • 8 years ago

Comment on attachment 8754784 [details]
MozReview Request: Bug 1208390 - Implement MediaStream.getTrackById(). r?jib, r?smaug

https://reviewboard.mozilla.org/r/54248/#review50934

	Jan-Ivar Bruaroey [:jib] (needinfo? me)
	Comment 5 • 8 years ago

Comment on attachment 8754782 [details]
MozReview Request: Bug 1208390 - Add test for MediaStream.getTrackById(). r?jib

https://reviewboard.mozilla.org/r/54244/#review50964

	Jan-Ivar Bruaroey [:jib] (needinfo? me)
	Comment 6 • 8 years ago

Comment on attachment 8754783 [details]
MozReview Request: Bug 1208390 - Use MediaStream.getTrackById() in some existing tests. r?jib

https://reviewboard.mozilla.org/r/54246/#review50968

::: dom/media/tests/mochitest/pc.js:1134
(Diff revision 1)
>      return this._pc.getRemoteStreams().some(stream => {
> -      return stream.getTracks().some(pcTrack => pcTrack.id == track.id);
> +      return !!stream.getTrackById(track.id);

Nit: An opportunity to remove {} and return ?

::: dom/media/tests/mochitest/pc.js:1481
(Diff revision 1)
>      var inputElem = from.localMediaElements[0];
>  
>      // As input we use the stream of |from|'s first available audio sender.
>      var inputSenderTracks = from._pc.getSenders().map(sn => sn.track);
>      var inputAudioStream = from._pc.getLocalStreams()
> -      .find(s => s.getAudioTracks().some(t => inputSenderTracks.some(t2 => t == t2)));
> +      .find(s => inputSenderTracks.some(t => t.kind == "audio" && !!s.getTrackById(t.id)));

!! is redundant here.

	Jan-Ivar Bruaroey [:jib] (needinfo? me)
	Comment 7 • 8 years ago

Comment on attachment 8754784 [details]
MozReview Request: Bug 1208390 - Implement MediaStream.getTrackById(). r?jib, r?smaug

https://reviewboard.mozilla.org/r/54248/#review50984

::: dom/media/DOMMediaStream.h:352
(Diff revision 1)
>    void GetAudioTracks(nsTArray<RefPtr<AudioStreamTrack> >& aTracks) const;
>    void GetVideoTracks(nsTArray<RefPtr<VideoStreamTrack> >& aTracks) const;
>    void GetTracks(nsTArray<RefPtr<MediaStreamTrack> >& aTracks) const;
> +  MediaStreamTrack* GetTrackById(const nsAString& aId);

Maybe const like the others here?

	Pulsebot
	Comment 8 • 8 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/dc94f1f82084
https://hg.mozilla.org/integration/mozilla-inbound/rev/54985370fcd2
https://hg.mozilla.org/integration/mozilla-inbound/rev/3737024731e6

	Andreas Pehrson [:pehrsons] Assignee
	Comment 9 • 8 years ago

Try push was here: https://treeherder.mozilla.org/#/jobs?repo=try&revision=4054d608608e

	Carsten Book [:Tomcat]
	Comment 10 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/dc94f1f82084
https://hg.mozilla.org/mozilla-central/rev/54985370fcd2
https://hg.mozilla.org/mozilla-central/rev/3737024731e6

	Eric Shepherd [:sheppy]
	Comment 11 • 8 years ago

Document created: https://developer.mozilla.org/en-US/docs/Web/API/MediaStreamTrack/getTrackById

Updated Firefox 49 for developers.