fix: Fix playback with Apple Media Keys polyfill detection (#7958)

Author: avelad

lib/dash/content_protection.js

@@ -203,7 +203,7 @@ shaka.dash.ContentProtection = class {
    * @return {string}
    */
   static getFairPlayLicenseUrl(element) {
-    if (shaka.util.Platform.isMediaKeysPolyfilled()) {
+    if (shaka.util.Platform.isMediaKeysPolyfilled('apple')) {
       throw new shaka.util.Error(
           shaka.util.Error.Severity.CRITICAL,
           shaka.util.Error.Category.MANIFEST,

lib/hls/hls_parser.js

@@ -4767,7 +4767,7 @@ shaka.hls.HlsParser = class {
           shaka.util.Error.Code.HLS_MSE_ENCRYPTED_MP2T_NOT_SUPPORTED);
     }
 
-    if (shaka.util.Platform.isMediaKeysPolyfilled()) {
+    if (shaka.util.Platform.isMediaKeysPolyfilled('apple')) {
       throw new shaka.util.Error(
           shaka.util.Error.Severity.CRITICAL,
           shaka.util.Error.Category.MANIFEST,
@@ -4957,7 +4957,7 @@ shaka.hls.HlsParser = class {
           shaka.util.Error.Code.HLS_MSE_ENCRYPTED_MP2T_NOT_SUPPORTED);
     }
 
-    if (shaka.util.Platform.isMediaKeysPolyfilled()) {
+    if (shaka.util.Platform.isMediaKeysPolyfilled('apple')) {
       throw new shaka.util.Error(
           shaka.util.Error.Severity.CRITICAL,
           shaka.util.Error.Category.MANIFEST,

lib/util/platform.js

@@ -705,16 +705,13 @@ shaka.util.Platform = class {
   }
 
   /**
-   * Returns true if MediaKeys is polyfilled
+   * Returns true if MediaKeys is polyfilled by the specified polyfill.
    *
-   * @param {string=} polyfillType
+   * @param {string} polyfillType
    * @return {boolean}
    */
   static isMediaKeysPolyfilled(polyfillType) {
-    if (polyfillType) {
-      return polyfillType === window.shakaMediaKeysPolyfill;
-    }
-    return !!window.shakaMediaKeysPolyfill;
+    return polyfillType === window.shakaMediaKeysPolyfill;
   }
 
 

lib/util/player_configuration.js

@@ -75,7 +75,7 @@ shaka.util.PlayerConfiguration = class {
       persistentSessionOnlinePlayback: false,
       persistentSessionsMetadata: [],
       initDataTransform: (initData, initDataType, drmInfo) => {
-        if (shaka.util.Platform.isMediaKeysPolyfilled() &&
+        if (shaka.util.Platform.isMediaKeysPolyfilled('apple') &&
             initDataType == 'skd') {
           const cert = drmInfo.serverCertificate;
           const contentId =

test/util/platform_unit.js

@@ -134,18 +134,6 @@ describe('Platform', () => {
       window.shakaMediaKeysPolyfill = shakaMediaKeysPolyfill;
     });
 
-    it('should return true if media keys are polyfilled', () => {
-      window.shakaMediaKeysPolyfill = 'webkit';
-      const result = shaka.util.Platform.isMediaKeysPolyfilled();
-      expect(result).toBe(true);
-    });
-
-    it('should return false if media keys are not polyfilled', () => {
-      window.shakaMediaKeysPolyfill = '';
-      const result = shaka.util.Platform.isMediaKeysPolyfilled();
-      expect(result).toBe(false);
-    });
-
     it('should return true with a matching polyfill type', () => {
       window.shakaMediaKeysPolyfill = 'webkit';
       const result = shaka.util.Platform.isMediaKeysPolyfilled('webkit');