fix(player): Refined support for HLS.js by not including Safari as pa…

…rt of the supported browsers
openplayerjs · Sep 14, 2018 · 9df28f7 · 9df28f7
1 parent 6be3400
commit 9df28f7
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/dist/openplayer.js b/dist/openplayer.js
@@ -662,7 +662,7 @@ exports.SUPPORTS_HLS = function () {
   var sourceBuffer = window.SourceBuffer || window.WebKitSourceBuffer;
   var isTypeSupported = mediaSource && typeof mediaSource.isTypeSupported === 'function' && mediaSource.isTypeSupported('video/mp4; codecs="avc1.42E01E,mp4a.40.2"');
   var sourceBufferValidAPI = !sourceBuffer || sourceBuffer.prototype && typeof sourceBuffer.prototype.appendBuffer === 'function' && typeof sourceBuffer.prototype.remove === 'function';
-  return !!isTypeSupported && !!sourceBufferValidAPI;
+  return !!isTypeSupported && !!sourceBufferValidAPI && !exports.IS_SAFARI;
 };
 
 /***/ }),

diff --git a/dist/openplayer.min.js b/dist/openplayer.min.js
diff --git a/src/js/utils/constants.ts b/src/js/utils/constants.ts
@@ -129,5 +129,8 @@ export const SUPPORTS_HLS = () => {
         (sourceBuffer.prototype &&
         typeof sourceBuffer.prototype.appendBuffer === 'function' &&
         typeof sourceBuffer.prototype.remove === 'function');
-    return !!isTypeSupported && !!sourceBufferValidAPI;
+
+    // Safari is still an exception since it has built-in HLS support; currently HLS.js
+    // is still in beta to support Safari
+    return !!isTypeSupported && !!sourceBufferValidAPI && !IS_SAFARI;
 };