Add ua-parser-js to enable reliable browser name and version parsing (#…

…4001)

package.json

@@ -62,7 +62,8 @@
         "fast-deep-equal": "2.0.1",
         "html-entities": "^1.2.1",
         "imsc": "^1.0.2",
-        "localforage": "^1.7.1"
+        "localforage": "^1.7.1",
+        "ua-parser-js": "^1.0.2"
     },
     "repository": {
         "type": "git",

src/core/Utils.js

@@ -35,6 +35,7 @@
  */
 
 import path from 'path-browserify'
+import { UAParser } from 'ua-parser-js'
 
 class Utils {
     static mixin(dest, source, copy) {
@@ -175,6 +176,17 @@ class Utils {
             return targetUrl
         }
     }
+
+    static parseUserAgent(ua = null) {
+        try {
+            const uaString = ua === null ? typeof navigator !== 'undefined' ? navigator.userAgent.toLowerCase() : '' : '';
+
+            return UAParser(uaString);
+        }
+        catch(e) {
+            return {};
+        }
+    }
 }
 
 export default Utils;

src/streaming/controllers/CatchupController.js

@@ -35,6 +35,7 @@ import Constants from '../constants/Constants';
 import MediaPlayerEvents from '../MediaPlayerEvents';
 import Events from '../../core/events/Events';
 import MetricsConstants from '../constants/MetricsConstants';
+import Utils from '../../core/Utils';
 
 function CatchupController() {
     const context = this.context;
@@ -115,8 +116,8 @@ function CatchupController() {
         isCatchupSeekInProgress = false;
 
         // Detect safari browser (special behavior for low latency streams)
-        const ua = typeof navigator !== 'undefined' ? navigator.userAgent.toLowerCase() : '';
-        const isSafari = /safari/.test(ua) && !/chrome/.test(ua);
+        const ua = Utils.parseUserAgent();
+        const isSafari = ua && ua.browser && ua.browser.name && ua.browser.name.toLowerCase() === 'safari';
         minPlaybackRateChange = isSafari ? 0.25 : 0.02;
     }