Merge pull request #1559 from timsu/190323-fix-remove-video

Don't call getUserMedia when audio and video are false
meetecho · Apr 1, 2019 · c7b6661 · c7b6661
2 parents 9318620 + 0015c97
commit c7b6661
Showing 1 changed file with 12 additions and 8 deletions.
diff --git a/html/janus.js b/html/janus.js
@@ -2295,14 +2295,18 @@ function Janus(gatewayCallbacks) {
 						video: (videoExist && !media.keepVideo) ? videoSupport : false
 					};
 					Janus.debug("getUserMedia constraints", gumConstraints);
-					navigator.mediaDevices.getUserMedia(gumConstraints)
-						.then(function(stream) {
-							pluginHandle.consentDialog(false);
-							streamsDone(handleId, jsep, media, callbacks, stream);
-						}).catch(function(error) {
-							pluginHandle.consentDialog(false);
-							callbacks.error({code: error.code, name: error.name, message: error.message});
-						});
+					if (!gumConstraints.audio && !gumConstraints.video) {
+						streamsDone(handleId, jsep, media, callbacks, stream);
+					} else {
+						navigator.mediaDevices.getUserMedia(gumConstraints)
+							.then(function(stream) {
+								pluginHandle.consentDialog(false);
+								streamsDone(handleId, jsep, media, callbacks, stream);
+							}).catch(function(error) {
+								pluginHandle.consentDialog(false);
+								callbacks.error({code: error.code, name: error.name, message: error.message});
+							});
+					}
 				})
 				.catch(function(error) {
 					pluginHandle.consentDialog(false);