From acb3bd7ad728b1cda0f8e08c082047210cf4b41f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sa=C3=BAl=20Ibarra=20Corretg=C3=A9?= Date: Wed, 27 Jun 2018 13:42:56 +0200 Subject: [PATCH] feat(BrowserSupport): remove PluginRequiredBrowser WebRTC plugin support has been axed, this is now dead code. --- react/features/app/functions.web.js | 20 ++------ .../components/PluginRequiredBrowser.js | 47 ------------------- .../unsupported-browser/components/index.js | 1 - 3 files changed, 3 insertions(+), 65 deletions(-) delete mode 100644 react/features/unsupported-browser/components/PluginRequiredBrowser.js diff --git a/react/features/app/functions.web.js b/react/features/app/functions.web.js index 86bd6a424..15f8eff7b 100644 --- a/react/features/app/functions.web.js +++ b/react/features/app/functions.web.js @@ -2,10 +2,7 @@ import { toState } from '../base/redux'; import { getDeepLinkingPage } from '../deep-linking'; -import { - PluginRequiredBrowser, - UnsupportedDesktopBrowser -} from '../unsupported-browser'; +import { UnsupportedDesktopBrowser } from '../unsupported-browser'; import { // eslint-disable-next-line camelcase @@ -30,19 +27,8 @@ const _INTERCEPT_COMPONENT_RULES = [ state => { const { webRTCReady } = state['features/base/lib-jitsi-meet']; - switch (typeof webRTCReady) { - case 'boolean': - if (webRTCReady === false) { - return Promise.resolve(UnsupportedDesktopBrowser); - } - break; - - case 'undefined': - // If webRTCReady is not set, then we cannot base a decision on it. - break; - - default: - return Promise.resolve(PluginRequiredBrowser); + if (webRTCReady === false) { + return Promise.resolve(UnsupportedDesktopBrowser); } return Promise.resolve(); diff --git a/react/features/unsupported-browser/components/PluginRequiredBrowser.js b/react/features/unsupported-browser/components/PluginRequiredBrowser.js deleted file mode 100644 index 2cb5ac415..000000000 --- a/react/features/unsupported-browser/components/PluginRequiredBrowser.js +++ /dev/null @@ -1,47 +0,0 @@ -/* @flow */ - -import React, { Component } from 'react'; - -import { CHROME, CHROMIUM, FIREFOX } from './browserLinks'; - -/** - * React component representing plugin installation required page. - * - * @class PluginRequiredBrowser - */ -export default class PluginRequiredBrowser extends Component<*> { - - /** - * Renders the component. - * - * @returns {ReactElement} - */ - render() { - const ns = 'unsupported-desktop-browser'; - const nsLink = `${ns}__link`; - - return ( -
-

- Your browser requires a plugin for this conversation. -

-

- Once you install the plugin, it will be possible for you to - have your conversation here. For the best experience, - however, we strongly recommend that you do that using - the  - Chrome,  - Chromium or  - Firefox browsers. -

-
- ); - } -} - diff --git a/react/features/unsupported-browser/components/index.js b/react/features/unsupported-browser/components/index.js index 142fbf841..fc0d8e0fd 100644 --- a/react/features/unsupported-browser/components/index.js +++ b/react/features/unsupported-browser/components/index.js @@ -1,3 +1,2 @@ -export { default as PluginRequiredBrowser } from './PluginRequiredBrowser'; export { default as UnsupportedDesktopBrowser } from './UnsupportedDesktopBrowser';