From 2ee5a92ef02ceb6fd6791b9835df813dacbef5dd Mon Sep 17 00:00:00 2001 From: George Politis Date: Mon, 22 Dec 2014 15:08:49 +0100 Subject: [PATCH] Adds comment to modules/RTC/MediaStream.js --- modules/RTC/MediaStream.js | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/modules/RTC/MediaStream.js b/modules/RTC/MediaStream.js index a981eaed3..cf1755d36 100644 --- a/modules/RTC/MediaStream.js +++ b/modules/RTC/MediaStream.js @@ -15,6 +15,17 @@ * @constructor */ function MediaStream(data, sid, ssrc, eventEmmiter, browser) { + + // XXX(gp) to minimize headaches in the future, we should build our + // abstractions around tracks and not streams. ORTC is track based API. + // Mozilla expects m-lines to represent media tracks. + // + // Practically, what I'm saying is that we should have a MediaTrack class + // and not a MediaStream class. + // + // Also, we should be able to associate multiple SSRCs with a MediaTrack as + // a track might have an associated RTX and FEC sources. + this.sid = sid; this.stream = data.stream; this.peerjid = data.peerjid;