Renames JitsiTrack.stop method and TRACK_STOPPED event. Fixes minor issues with TRACK_REMOVED event.

JitsiConference.js

@@ -12,6 +12,7 @@ var JitsiParticipant = require("./JitsiParticipant");
 var Statistics = require("./modules/statistics/statistics");
 var JitsiDTMFManager = require('./modules/DTMF/JitsiDTMFManager');
 var JitsiTrackEvents = require("./JitsiTrackEvents");
+var JitsiTrackErrors = require("./JitsiTrackErrors");
 var Settings = require("./modules/settings/Settings");
 
 /**
@@ -298,6 +299,10 @@ JitsiConference.prototype.setSubject = function (subject) {
  * and there is already another video track in the conference.
  */
 JitsiConference.prototype.addTrack = function (track) {
+    if(track.disposed)
+    {
+        throw new Error(JitsiTrackErrors.TRACK_IS_DISPOSED);
+    }
     if (track.isVideoTrack()) {
         if (this.rtc.getLocalVideoStream()) {
             throw new Error("cannot add second video track to the conference");
@@ -336,12 +341,9 @@ JitsiConference.prototype.addTrack = function (track) {
             }
 
             track.muteHandler = this._fireMuteChangeEvent.bind(this, track);
-            track.stopHandler = this.removeTrack.bind(this, track);
             track.audioLevelHandler = this._fireAudioLevelChangeEvent.bind(this);
             track.addEventListener(JitsiTrackEvents.TRACK_MUTE_CHANGED,
                                    track.muteHandler);
-            track.addEventListener(JitsiTrackEvents.TRACK_STOPPED,
-                                   track.stopHandler);
             track.addEventListener(JitsiTrackEvents.TRACK_AUDIO_LEVEL_CHANGED,
                                    track.audioLevelHandler);
             //FIXME: This dependacy is not necessary. This is quick fix.
@@ -382,6 +384,10 @@ JitsiConference.prototype._fireMuteChangeEvent = function (track) {
  * @returns {Promise}
  */
 JitsiConference.prototype.removeTrack = function (track) {
+    if(track.disposed)
+    {
+        throw new Error(JitsiTrackErrors.TRACK_IS_DISPOSED);
+    }
     if(!this.room){
         if(this.rtc) {
             this.rtc.removeLocalStream(track);
@@ -397,8 +403,6 @@ JitsiConference.prototype.removeTrack = function (track) {
             this.rtc.removeLocalStream(track);
             track.removeEventListener(JitsiTrackEvents.TRACK_MUTE_CHANGED,
                 track.muteHandler);
-            track.removeEventListener(JitsiTrackEvents.TRACK_STOPPED,
-                track.stopHandler);
             track.removeEventListener(JitsiTrackEvents.TRACK_AUDIO_LEVEL_CHANGED,
                 track.audioLevelHandler);
             this.room.removeListener(XMPPEvents.SENDRECV_STREAMS_CHANGED,
@@ -868,7 +872,7 @@ function setupListeners(conference) {
                 for(var i = 0; i < tracks.length; i++) {
                     if(tracks[i] && tracks[i].stream &&
                         RTC.getStreamID(tracks[i].stream) == streamId){
-                        var track = participant._tracks.splice(i, 1);
+                        var track = participant._tracks.splice(i, 1)[0];
                         conference.eventEmitter.emit(
                             JitsiConferenceEvents.TRACK_REMOVED, track);
                         return;

JitsiMeetJS.js

@@ -92,7 +92,7 @@ var LibJitsiMeet = {
                             Statistics.startLocalStats(mStream,
                                 track.setAudioLevel.bind(track));
                             track.addEventListener(
-                                JitsiTrackEvents.TRACK_STOPPED,
+                                JitsiTrackEvents.LOCAL_TRACK_STOPPED,
                                 function(){
                                     Statistics.stopLocalStats(mStream);
                                 });

JitsiTrackErrors.js

@@ -23,5 +23,6 @@ module.exports = {
     },
     UNSUPPORTED_RESOLUTION: "gum.unsupported_resolution",
     FIREFOX_EXTENSION_NEEDED: "gum.firefox_extension_needed",
-    GENERAL: "gum.general"
+    GENERAL: "gum.general",
+    TRACK_IS_DISPOSED: "track.track_is_disposed"
 };

JitsiTrackEvents.js

@@ -10,7 +10,7 @@ var JitsiTrackEvents = {
     /**
      * The media track was removed to the conference.
      */
-    TRACK_STOPPED: "track.stopped",
+    LOCAL_TRACK_STOPPED: "track.stopped",
     /**
      * The video type("camera" or "desktop") of the track was changed.
      */

doc/API.md

@@ -106,6 +106,10 @@ JitsiMeetJS.setLogLevel(JitsiMeetJS.logLevels.ERROR);
         - CONNECTION_DISCONNECTED - indicates that we are disconnected.
         - WRONG_STATE - indicates that the user has performed action that can't be executed because the connection is in wrong state.
 
+    3. tracks
+        - LOCAL_TRACK_STOPPED - indicates that a local track was stopped. This
+        event can be fired when ```dispose()``` method is called or for other reasons.
+
 * ```JitsiMeetJS.errors``` - JS object that contains all errors used by the API. You can use that object to check the reported errors from the API
     We have two error types - connection and conference. You can access the events with the following code ```JitsiMeetJS.errors.<error_type>.<error_name>```.
     For example if you want to use the conference event that is fired when somebody leave conference you can use the following code - ```JitsiMeetJS.errors.conference.PASSWORD_REQUIRED```.
@@ -319,7 +323,7 @@ We have the following methods for controling the tracks:
 
 6. detach(container) - removes the track from the container.
 
-7. stop() - stop sending the track to the other participants in the conference. Returns Promise.
+7. dispose() - disposes the track. If the track is added to a conference the track will be removed. Returns Promise.
 
    Note: This method is implemented only for the local tracks.
 

doc/example/example.js

@@ -32,7 +32,7 @@ function onLocalTracks(tracks)
             function () {
                 console.log("local track muted");
             });
-        localTracks[i].addEventListener(JitsiMeetJS.events.track.TRACK_STOPPED,
+        localTracks[i].addEventListener(JitsiMeetJS.events.track.LOCAL_TRACK_STOPPED,
             function () {
                 console.log("local track stoped");
             });
@@ -67,7 +67,7 @@ function onRemoteTrack(track) {
         function () {
             console.log("remote track muted");
         });
-    track.addEventListener(JitsiMeetJS.events.track.TRACK_STOPPED,
+    track.addEventListener(JitsiMeetJS.events.track.LOCAL_TRACK_STOPPED,
         function () {
             console.log("remote track stoped");
         });
@@ -159,7 +159,7 @@ var isVideo = true;
 function switchVideo() {
     isVideo = !isVideo;
     if(localTracks[1]) {
-        localTracks[1].stop();
+        localTracks[1].dispose();
         localTracks.pop();
     }
     JitsiMeetJS.createLocalTracks({devices: isVideo? ["video"] : ["desktop"]}).
@@ -169,7 +169,7 @@ function switchVideo() {
                 function () {
                     console.log("local track muted");
                 });
-            localTracks[1].addEventListener(JitsiMeetJS.events.track.TRACK_STOPPED,
+            localTracks[1].addEventListener(JitsiMeetJS.events.track.LOCAL_TRACK_STOPPED,
                 function () {
                     console.log("local track stoped");
                 });

modules/RTC/JitsiLocalTrack.js

@@ -17,13 +17,14 @@ function JitsiLocalTrack(stream, videoType,
     this.deviceId = deviceId;
     this.startMuted = false;
     this.ssrc = null;
+    this.disposed = false;
     //FIXME: This dependacy is not necessary.
     this.conference = null;
     JitsiTrack.call(this, null, stream,
         function () {
             if(!this.dontFireRemoveEvent)
                 this.eventEmitter.emit(
-                    JitsiTrackEvents.TRACK_STOPPED);
+                    JitsiTrackEvents.LOCAL_TRACK_STOPPED);
             this.dontFireRemoveEvent = false;
         }.bind(this));
     this.initialMSID = this.getMSID();
@@ -130,7 +131,7 @@ JitsiLocalTrack.prototype._setMute = function (mute) {
  * NOTE: Works for local tracks only.
  * @returns {Promise}
  */
-JitsiLocalTrack.prototype.stop = function () {
+JitsiLocalTrack.prototype.dispose = function () {
     var promise = Promise.resolve();
 
     if (this.conference){
@@ -141,6 +142,7 @@ JitsiLocalTrack.prototype.stop = function () {
         RTCUtils.stopMediaStream(this.stream);
         this.detach();
     }
+    this.disposed = true;
 
     return promise;
 };

modules/RTC/JitsiRemoteTrack.js

@@ -12,9 +12,7 @@ var JitsiTrackEvents = require("../../JitsiTrackEvents");
  */
 function JitsiRemoteTrack(RTC, data, sid, ssrc) {
     JitsiTrack.call(this, RTC, data.stream,
-        function () {
-            this.eventEmitter.emit(JitsiTrackEvents.TRACK_STOPPED);
-        }.bind(this), data.jitsiTrackType);
+        function () {}, data.jitsiTrackType);
     this.rtc = RTC;
     this.sid = sid;
     this.stream = data.stream;
@@ -89,6 +87,6 @@ JitsiRemoteTrack.prototype._setVideoType = function (type) {
     this.eventEmitter.emit(JitsiTrackEvents.TRACK_VIDEOTYPE_CHANGED, type);
 }
 
-delete JitsiRemoteTrack.prototype.stop;
+delete JitsiRemoteTrack.prototype.dispose;
 
 module.exports = JitsiRemoteTrack;

modules/RTC/JitsiTrack.js

@@ -220,10 +220,10 @@ JitsiTrack.prototype.detach = function (container) {
 }
 
 /**
- * Stops sending the media track. And removes it from the HTML.
+ * Dispose sending the media track. And removes it from the HTML.
  * NOTE: Works for local tracks only.
  */
-JitsiTrack.prototype.stop = function () {
+JitsiTrack.prototype.dispose = function () {
 }
 
 /**