From 21957c8bf2d1ee0b1c66eb887e7ac5c2cb93ef14 Mon Sep 17 00:00:00 2001
From: paweldomas <pawel.domas@jitsi.org>
Date: Mon, 10 Jul 2017 16:32:50 +0200
Subject: [PATCH] fix(conference): disable video icon

Explicitly calls 'updateVideoIconEnabled' in case both audio and video
permission were not granted and device list changed callbacks will not
be executed (check in lib-jitsi-meet that it requires GUM call to
succeed at least once for device list changed events).
---
 conference.js | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/conference.js b/conference.js
index c60854367..d46c43dba 100644
--- a/conference.js
+++ b/conference.js
@@ -629,6 +629,13 @@ export default {
                     // the video later on, even if joined without it.
                     this.videoMuted = true;
                     APP.UI.setVideoMuted(this.getMyUserId(), this.videoMuted);
+                    // FIXME this is a workaround for the situation where
+                    // both audio and video permissions are rejected initially
+                    // and the callback from _initDeviceList will never be
+                    // executed (GUM not initialized - check lib-jitsi-meet).
+                    // The goal here is to disable the video icon in case no
+                    // video permissions were granted.
+                    this.updateVideoIconEnabled();
                 }
 
                 this._initDeviceList();