Browse Source

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).
j8
paweldomas 8 years ago
parent
commit
21957c8bf2
1 changed files with 7 additions and 0 deletions
  1. 7
    0
      conference.js

+ 7
- 0
conference.js View File

@@ -629,6 +629,13 @@ export default {
629 629
                     // the video later on, even if joined without it.
630 630
                     this.videoMuted = true;
631 631
                     APP.UI.setVideoMuted(this.getMyUserId(), this.videoMuted);
632
+                    // FIXME this is a workaround for the situation where
633
+                    // both audio and video permissions are rejected initially
634
+                    // and the callback from _initDeviceList will never be
635
+                    // executed (GUM not initialized - check lib-jitsi-meet).
636
+                    // The goal here is to disable the video icon in case no
637
+                    // video permissions were granted.
638
+                    this.updateVideoIconEnabled();
632 639
                 }
633 640
 
634 641
                 this._initDeviceList();

Loading…
Cancel
Save