Fixing issue when both mic and cam are stopped

front/src/WebRtc/MediaManager.ts

@@ -7,9 +7,9 @@ const videoConstraint: boolean|MediaTrackConstraints = {
     facingMode: "user"
 };
 
-type UpdatedLocalStreamCallback = (media: MediaStream) => void;
+export type UpdatedLocalStreamCallback = (media: MediaStream|null) => void;
-type StartScreenSharingCallback = (media: MediaStream) => void;
+export type StartScreenSharingCallback = (media: MediaStream) => void;
-type StopScreenSharingCallback = (media: MediaStream) => void;
+export type StopScreenSharingCallback = (media: MediaStream) => void;
 
 // TODO: Split MediaManager in 2 classes: MediaManagerUI (in charge of HTML) and MediaManager (singleton in charge of the camera only)
 // TODO: verify that microphone event listeners are not triggered plenty of time NOW (since MediaManager is created many times!!!!)
@@ -109,7 +109,7 @@ export class MediaManager {
         this.updatedLocalStreamCallBacks.delete(callback);
     }
 
-    private triggerUpdatedLocalStreamCallbacks(stream: MediaStream): void {
+    private triggerUpdatedLocalStreamCallbacks(stream: MediaStream|null): void {
         for (const callback of this.updatedLocalStreamCallBacks) {
             callback(stream);
         }
@@ -142,16 +142,20 @@ export class MediaManager {
         });
     }
 
-    private disableCamera() {
+    private async disableCamera() {
         this.cinemaClose.style.display = "block";
         this.cinema.style.display = "none";
         this.cinemaBtn.classList.add("disabled");
         this.constraintsMedia.video = false;
         this.myCamVideo.srcObject = null;
         this.stopCamera();
-        this.getCamera().then((stream) => {
+
+        if (this.constraintsMedia.audio !== false) {
+            const stream = await this.getCamera();
             this.triggerUpdatedLocalStreamCallbacks(stream);
-        });
+        } else {
+            this.triggerUpdatedLocalStreamCallbacks(null);
+        }
     }
 
     private enableMicrophone() {
@@ -159,20 +163,25 @@ export class MediaManager {
         this.microphone.style.display = "block";
         this.microphoneBtn.classList.remove("disabled");
         this.constraintsMedia.audio = true;
+
         this.getCamera().then((stream) => {
             this.triggerUpdatedLocalStreamCallbacks(stream);
         });
     }
 
-    private disableMicrophone() {
+    private async disableMicrophone() {
         this.microphoneClose.style.display = "block";
         this.microphone.style.display = "none";
         this.microphoneBtn.classList.add("disabled");
         this.constraintsMedia.audio = false;
         this.stopMicrophone();
-        this.getCamera().then((stream) => {
+
+        if (this.constraintsMedia.video !== false) {
+            const stream = await this.getCamera();
             this.triggerUpdatedLocalStreamCallbacks(stream);
-        });
+        } else {
+            this.triggerUpdatedLocalStreamCallbacks(null);
+        }
     }
 
     private enableScreenSharing() {

front/src/WebRtc/ScreenSharingPeer.ts

@@ -70,7 +70,7 @@ export class ScreenSharingPeer extends Peer {
     }
 
     private sendWebrtcScreenSharingSignal(data: unknown) {
-        console.log("sendWebrtcScreenSharingSignal", data);
+        //console.log("sendWebrtcScreenSharingSignal", data);
         try {
             this.connection.sendWebrtcScreenSharingSignal(data, this.userId);
         }catch (e) {
@@ -82,8 +82,8 @@ export class ScreenSharingPeer extends Peer {
      * Sends received stream to screen.
      */
     private stream(stream?: MediaStream) {
-        console.log(`ScreenSharingPeer::stream => ${this.userId}`, stream);
+        //console.log(`ScreenSharingPeer::stream => ${this.userId}`, stream);
-        console.log(`stream => ${this.userId} => `, stream);
+        //console.log(`stream => ${this.userId} => `, stream);
         if(!stream){
             mediaManager.removeActiveScreenSharingVideo(this.userId);
             this.isReceivingStream = false;

front/src/WebRtc/SimplePeer.ts

@@ -4,7 +4,12 @@ import {
     WebRtcSignalReceivedMessageInterface,
     WebRtcStartMessageInterface
 } from "../Connection";
-import { mediaManager } from "./MediaManager";
+import {
+    mediaManager,
+    StartScreenSharingCallback,
+    StopScreenSharingCallback,
+    UpdatedLocalStreamCallback
+} from "./MediaManager";
 import * as SimplePeerNamespace from "simple-peer";
 import {ScreenSharingPeer} from "./ScreenSharingPeer";
 import {VideoPeer} from "./VideoPeer";
@@ -32,9 +37,9 @@ export class SimplePeer {
 
     private PeerScreenSharingConnectionArray: Map<string, ScreenSharingPeer> = new Map<string, ScreenSharingPeer>();
     private PeerConnectionArray: Map<string, VideoPeer> = new Map<string, VideoPeer>();
-    private readonly sendLocalVideoStreamCallback: (media: MediaStream) => void;
+    private readonly sendLocalVideoStreamCallback: UpdatedLocalStreamCallback;
-    private readonly sendLocalScreenSharingStreamCallback: (media: MediaStream) => void;
+    private readonly sendLocalScreenSharingStreamCallback: StartScreenSharingCallback;
-    private readonly stopLocalScreenSharingStreamCallback: (media: MediaStream) => void;
+    private readonly stopLocalScreenSharingStreamCallback: StopScreenSharingCallback;
     private readonly peerConnectionListeners: Array<PeerConnectionListener> = new Array<PeerConnectionListener>();
 
     constructor(Connection: Connection, WebRtcRoomId: string = "test-webrtc") {
@@ -326,9 +331,9 @@ export class SimplePeer {
     }
 
     public sendLocalVideoStream(){
-        this.Users.forEach((user: UserSimplePeerInterface) => {
+        for (const user of this.Users) {
             this.pushVideoToRemoteUser(user.userId);
-        })
+        }
     }
 
     /**

front/src/WebRtc/VideoPeer.ts

@@ -85,7 +85,7 @@ export class VideoPeer extends Peer {
      * Sends received stream to screen.
      */
     private stream(stream?: MediaStream) {
-        console.log(`VideoPeer::stream => ${this.userId}`, stream);
+        //console.log(`VideoPeer::stream => ${this.userId}`, stream);
         if(!stream){
             mediaManager.disabledVideoByUserId(this.userId);
             mediaManager.disabledMicrophoneByUserId(this.userId);