Browse Source

Adds comment to modules/RTC/MediaStream.js

master
George Politis 11 years ago
parent
commit
2ee5a92ef0
1 changed files with 11 additions and 0 deletions
  1. 11
    0
      modules/RTC/MediaStream.js

+ 11
- 0
modules/RTC/MediaStream.js View File

@@ -15,6 +15,17 @@
15 15
  * @constructor
16 16
  */
17 17
 function MediaStream(data, sid, ssrc, eventEmmiter, browser) {
18
+
19
+    // XXX(gp) to minimize headaches in the future, we should build our
20
+    // abstractions around tracks and not streams. ORTC is track based API.
21
+    // Mozilla expects m-lines to represent media tracks.
22
+    //
23
+    // Practically, what I'm saying is that we should have a MediaTrack class
24
+    // and not a MediaStream class.
25
+    //
26
+    // Also, we should be able to associate multiple SSRCs with a MediaTrack as
27
+    // a track might have an associated RTX and FEC sources.
28
+
18 29
     this.sid = sid;
19 30
     this.stream = data.stream;
20 31
     this.peerjid = data.peerjid;

Loading…
Cancel
Save