openvidu-browser: bug fix when checking system requirements on motoro…

…la edge devices
neilhzx · May 19, 2023 · 58c15b0 · 58c15b0
1 parent ba0e736
commit 58c15b0
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git 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
@@ -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
      */