From 58c15b0e3c9880df30ce32b1ef7563e48e299998 Mon Sep 17 00:00:00 2001 From: pabloFuente Date: Fri, 19 May 2023 13:39:19 +0200 Subject: [PATCH] openvidu-browser: bug fix when checking system requirements on motorola edge devices --- openvidu-browser/src/OpenVidu/OpenVidu.ts | 4 +++- .../src/OpenViduInternal/Utils/Platform.ts | 14 +++++++++++++- 2 files changed, 16 insertions(+), 2 deletions(-) diff --git a/openvidu-browser/src/OpenVidu/OpenVidu.ts b/openvidu-browser/src/OpenVidu/OpenVidu.ts index f8eb2c55..6d1361c9 100644 --- a/openvidu-browser/src/OpenVidu/OpenVidu.ts +++ b/openvidu-browser/src/OpenVidu/OpenVidu.ts @@ -348,7 +348,9 @@ export class OpenVidu { platform.isSafariBrowser() || platform.isAndroidBrowser() || // Android WebView & Ionic apps for Android platform.isElectron() || - platform.isNodeJs() + platform.isNodeJs() || + // TODO: remove when updating platform detection library + platform.isMotorolaEdgeDevice() ); } diff --git a/openvidu-browser/src/OpenViduInternal/Utils/Platform.ts b/openvidu-browser/src/OpenViduInternal/Utils/Platform.ts index 3281ea7f..7cf6e301 100644 --- a/openvidu-browser/src/OpenViduInternal/Utils/Platform.ts +++ b/openvidu-browser/src/OpenViduInternal/Utils/Platform.ts @@ -101,6 +101,14 @@ export class PlatformUtils { return platform.name === 'Samsung Internet Mobile' || platform.name === 'Samsung Internet'; } + // TODO: This method exists to overcome bug https://github.com/bestiejs/platform.js/issues/184 + /** + * @hidden + */ + public isMotorolaEdgeDevice(): boolean { + return platform.product?.toLowerCase().includes('motorola edge') || false; + } + /** * @hidden */ @@ -168,10 +176,14 @@ export class PlatformUtils { this.isSamsungBrowser() || this.isIonicAndroid() || this.isIonicIos() || - this.isElectron() + this.isElectron() || + // TODO: remove when possible + this.isMotorolaEdgeDevice() ); } + + /** * @hidden */