Merge pull request #217 from theurere/cleanup-screensharing

Improve handling of peercall event remoteStreamAdded

static/js/directives/screenshare.js

@@ -95,9 +95,11 @@ define(['jquery', 'underscore', 'text!partials/screenshare.html', 'text!partials
 				mediaStream.webrtc.doSubscribeScreenshare(from, token, {
 					created: function(peerscreenshare) {
 						peerscreenshare.e.on("remoteStreamAdded", function(event, stream) {
-							$scope.$apply(function(scope) {
-								scope.addRemoteStream(stream, peerscreenshare);
-							});
+							if (stream) {
+								$scope.$apply(function(scope) {
+									scope.addRemoteStream(stream, peerscreenshare);
+								});
+							}
 						});
 						peerscreenshare.e.on("remoteStreamRemoved", function(event, stream) {
 							safeApply($scope, function(scope) {

static/js/services/videowaiter.js

@@ -36,7 +36,7 @@ define(["underscore"], function(_) {
 				}
 				return;
 			}
-			var videoTracks = stream.getVideoTracks();
+			var videoTracks = stream && stream.getVideoTracks() || [];
 			//console.log("wait for video", videoTracks.length, video.currentTime, video.videoHeight, video);
 			if (videoTracks.length === 0 && this.count >= 10) {
 				cb(false, video, stream);