Skip to content

Commit

Permalink
fix(FEC-9379): Edge chromium should use playready when exist (#274)
Browse files Browse the repository at this point in the history
chromiom Edge is supportted widevine as well so added default config for Edge to prefer playing Playready(Hardware DRM)
  • Loading branch information
Yuvalke authored Oct 31, 2019
1 parent 1ca1b5d commit 6b87274
Showing 1 changed file with 16 additions and 1 deletion.
17 changes: 16 additions & 1 deletion src/common/utils/setup-helpers.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
// @flow
import {setDefaultAnalyticsPlugin} from 'player-defaults';
import {Env, TextStyle, Utils, setCapabilities, EngineType} from '@playkit-js/playkit-js';
import {Env, TextStyle, Utils, setCapabilities, EngineType, DrmScheme} from '@playkit-js/playkit-js';
import {ValidationErrorType} from './validation-error';
import StorageManager from '../storage/storage-manager';
import type {LogLevelObject} from './logger';
Expand Down Expand Up @@ -281,6 +281,7 @@ function getDefaultOptions(options: PartialKPOptionsObject): KPOptionsObject {
checkNativeTextTracksSupport(defaultOptions);
setDefaultAnalyticsPlugin(defaultOptions);
configureLGTVDefaultOptions(defaultOptions);
configureEdgeDRMDefaultOptions(defaultOptions);
configureIMADefaultOptions(defaultOptions);
configureDAIDefaultOptions(defaultOptions);
configureBumperDefaultOptions(defaultOptions);
Expand Down Expand Up @@ -394,6 +395,20 @@ function configureLGTVDefaultOptions(options: KPOptionsObject): void {
}
}

/**
* prefer Playready in edge - from chromium version of edge Widevine is option as well
* @private
* @param {KPOptionsObject} options - kaltura player options
* @returns {void}
*/
function configureEdgeDRMDefaultOptions(options: KPOptionsObject): void {
if (Env.browser.name === 'Edge') {
const keySystem = Utils.Object.getPropertyPath(options, 'drm.keySystem');
if (typeof keySystem !== 'boolean') {
options = Utils.Object.createPropertyPath(options, 'drm.keySystem', DrmScheme.PLAYREADY);
}
}
}
/**
* Sets default config option for ima plugin
* @private
Expand Down

0 comments on commit 6b87274

Please sign in to comment.