From 142566dfde2fdaa92f940c152b9427ce8da86a7c Mon Sep 17 00:00:00 2001
From: Gregoire Parant <g.parant@thecodingmachine.com>
Date: Tue, 10 Nov 2020 14:03:29 +0100
Subject: [PATCH] Fix error get media stream camera

Fix error camera when user have not cam on computer
---
 front/src/WebRtc/MediaManager.ts | 57 +++++++++++++++++++++-----------
 1 file changed, 37 insertions(+), 20 deletions(-)

diff --git a/front/src/WebRtc/MediaManager.ts b/front/src/WebRtc/MediaManager.ts
index ab6d9a3b..fa703fbe 100644
--- a/front/src/WebRtc/MediaManager.ts
+++ b/front/src/WebRtc/MediaManager.ts
@@ -38,6 +38,8 @@ export class MediaManager {
     private cinemaBtn: HTMLDivElement;
     private monitorBtn: HTMLDivElement;
 
+    private hasCamera = true;
+
     constructor() {
 
         this.myCamVideo = this.getElementByIdOrFail<HTMLVideoElement>('myCamVideo');
@@ -136,6 +138,9 @@ export class MediaManager {
     }
 
     public enableCamera() {
+        if(!this.hasCamera){
+            return;
+        }
         this.cinemaClose.style.display = "none";
         this.cinemaBtn.classList.remove("disabled");
         this.cinema.style.display = "block";
@@ -146,13 +151,7 @@ export class MediaManager {
     }
 
     public 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.disabledCameraView();
         if (this.constraintsMedia.audio !== false) {
             const stream = await this.getCamera();
             this.triggerUpdatedLocalStreamCallbacks(stream);
@@ -161,6 +160,15 @@ export class MediaManager {
         }
     }
 
+    private disabledCameraView(){
+        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();
+    }
+
     public enableMicrophone() {
         this.microphoneClose.style.display = "none";
         this.microphone.style.display = "block";
@@ -267,24 +275,33 @@ export class MediaManager {
             }
         }
 
-        try {
-            const stream = await navigator.mediaDevices.getUserMedia(this.constraintsMedia);
+        return this.getLocalStream().catch(() => {
+            console.info('Error get camera, trying with video option at null');
+            this.disabledCameraView();
+            return this.getLocalStream().then((stream : MediaStream) => {
+                this.hasCamera = false;
+                return stream;
+            }).catch((err) => {
+                console.info("error get media ", this.constraintsMedia.video, this.constraintsMedia.audio, err);
+                throw err;
+            });
+        });
 
+        //TODO resize remote cam
+        /*console.log(this.localStream.getTracks());
+        let videoMediaStreamTrack =  this.localStream.getTracks().find((media : MediaStreamTrack) => media.kind === "video");
+        let {width, height} = videoMediaStreamTrack.getSettings();
+        console.info(`${width}x${height}`); // 6*/
+    }
+
+    private getLocalStream() : Promise<MediaStream> {
+        return navigator.mediaDevices.getUserMedia(this.constraintsMedia).then((stream : MediaStream) => {
             this.localStream = stream;
             this.myCamVideo.srcObject = this.localStream;
-
             return stream;
-
-            //TODO resize remote cam
-            /*console.log(this.localStream.getTracks());
-            let videoMediaStreamTrack =  this.localStream.getTracks().find((media : MediaStreamTrack) => media.kind === "video");
-            let {width, height} = videoMediaStreamTrack.getSettings();
-            console.info(`${width}x${height}`); // 6*/
-        } catch (err) {
-            console.info("error get media ", this.constraintsMedia.video, this.constraintsMedia.audio, err);
-            this.localStream = null;
+        }).catch((err: Error) => {
             throw err;
-        }
+        });
     }
 
     /**