Skip to content

Commit

Permalink
fix(FEC-11126): upgrade shaka to 3.0.10 (#139)
Browse files Browse the repository at this point in the history
updated shaka to 3.0.10
moved useShakaTextTrackDisplay from playback.options.playback.options.html5.dash to text
added useShakaTextTrackDisplay in _getParsedTextTracks when creating TextTrack and kind is an empty string
  • Loading branch information
RoyBregman committed Apr 12, 2021
1 parent c3c523b commit f6268a6
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 8 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@
"mocha": "^8.0.1",
"mocha-cli": "^1.0.1",
"prettier": "^2.0.5",
"shaka-player": "3.0.8",
"shaka-player": "3.0.10",
"sinon": "^9.0.2",
"sinon-chai": "^3.5.0",
"standard-version": "^6.0.1",
Expand Down
10 changes: 7 additions & 3 deletions src/dash-adapter.js
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,6 @@ export default class DashAdapter extends BaseMediaSourceAdapter {
* @private
*/
_thumbnailController: ?DashThumbnailController;

/**
* Factory method to create media source adapter.
* @function createAdapter
Expand All @@ -202,6 +201,9 @@ export default class DashAdapter extends BaseMediaSourceAdapter {
if (Utils.Object.hasPropertyPath(config, 'text.useNativeTextTrack')) {
adapterConfig.textTrackVisibile = Utils.Object.getPropertyPath(config, 'text.useNativeTextTrack');
}
if (Utils.Object.hasPropertyPath(config, 'text.useShakaTextTrackDisplay')) {
adapterConfig.useShakaTextTrackDisplay = Utils.Object.getPropertyPath(config, 'text.useShakaTextTrackDisplay');
}
if (Utils.Object.hasPropertyPath(config, 'sources.options')) {
const options = config.sources.options;
adapterConfig.forceRedirectExternalStreams = options.forceRedirectExternalStreams;
Expand Down Expand Up @@ -358,7 +360,7 @@ export default class DashAdapter extends BaseMediaSourceAdapter {
shaka.polyfill.installAll();
this._shaka = new shaka.Player();
//render text tracks to our own container
if (this._config.shakaConfig.useShakaTextTrackDisplay) {
if (this._config.useShakaTextTrackDisplay) {
this._shaka.setVideoContainer(Utils.Dom.getElementBySelector('.playkit-subtitles'));
}
this._maybeSetFilters();
Expand Down Expand Up @@ -856,8 +858,10 @@ export default class DashAdapter extends BaseMediaSourceAdapter {
let parsedTracks = [];
if (textTracks) {
for (let i = 0; i < textTracks.length; i++) {
let kind = textTracks[i].kind ? textTracks[i].kind + 's' : '';
kind = kind === '' && this._config.useShakaTextTrackDisplay ? 'captions' : kind;
let settings = {
kind: textTracks[i].kind ? textTracks[i].kind + 's' : '',
kind: kind,
active: false,
label: textTracks[i].label,
language: textTracks[i].language,
Expand Down
8 changes: 4 additions & 4 deletions yarn.lock
Original file line number Diff line number Diff line change
Expand Up @@ -7340,10 +7340,10 @@ sha.js@^2.4.0, sha.js@^2.4.8:
inherits "^2.0.1"
safe-buffer "^5.0.1"

shaka-player@3.0.8:
version "3.0.8"
resolved "https://registry.yarnpkg.com/shaka-player/-/shaka-player-3.0.8.tgz#32e4c321b80012bca9db9a20b7ec63eefd7df512"
integrity sha512-4nROuGUhmtOTERWVO31pp/hvtdCy/kW/Ys6KQwqRhFeCsZsU1/8768dJUyXKqNnbYybKIvuv6LqO8egkXy6TjA==
shaka-player@3.0.10:
version "3.0.10"
resolved "https://registry.yarnpkg.com/shaka-player/-/shaka-player-3.0.10.tgz#f5cca848df54694f1aa77097f990881da11dbf68"
integrity sha512-kr1OFbihAmsmtsb/QhyDfxkt05wPAdreiW8vF86RN95zxpp+J/thEcTjRrH0E9dkBwPwS4HVv+mm096uqtEMvA==
dependencies:
eme-encryption-scheme-polyfill "^2.0.1"

Expand Down

0 comments on commit f6268a6

Please sign in to comment.