feat(ShareControl): disable share screen on mobile browsers and Inter…

…net Explorer browser
webex · Mar 23, 2022 · 454db60 · 454db60
1 parent 04de0c1
commit 454db60
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/jest.setup.js b/jest.setup.js
@@ -35,6 +35,7 @@ global.MediaStream = jest.fn(function MockMediaStream(tracksOrStream = []) {
 global.navigator = {
   mediaDevices: {
     enumerateDevices: () => Promise.resolve(mockDevices),
+    getDisplayMedia: () => Promise.resolve(new MediaStream()),
   },
   userAgent: ''
 };

diff --git a/src/MeetingsSDKAdapter/controls/ShareControl.js b/src/MeetingsSDKAdapter/controls/ShareControl.js
@@ -1,4 +1,4 @@
-import {Observable} from 'rxjs';
+import {Observable, of} from 'rxjs';
 import {distinctUntilChanged, map, tap} from 'rxjs/operators';
 import {MeetingControlState} from '@webex/component-adapter-interfaces';
 import logger from '../../logger';
@@ -47,8 +47,18 @@ export default class ShareControl extends MeetingControl {
       text: 'Stop sharing',
       tooltip: 'Stop sharing content',
     };
+    const notSupported = {
+      ID: this.ID,
+      type: 'TOGGLE',
+      state: MeetingControlState.DISABLED,
+      icon: 'share-screen-presence-stroke',
+      text: 'Start sharing',
+      tooltip: 'Share screen not supported',
+    };
+
+    const browserSupport = navigator.mediaDevices && navigator.mediaDevices.getDisplayMedia;
 
-    return this.adapter.getMeeting(meetingID).pipe(
+    return !browserSupport ? of(notSupported) : this.adapter.getMeeting(meetingID).pipe(
       map(({localShare: {stream}}) => (stream ? active : inactive)),
       distinctUntilChanged(),
       tap((display) => logger.debug('MEETING', meetingID, 'ShareControl::display()', ['emitting', display])),