Skip to content

Commit

Permalink
fix(FEC-10404): media doesn't work properly on Safari browser - no vi…
Browse files Browse the repository at this point in the history
…deo displayed - Regression bug (#46)

Due to FEC-10057 (move the plugin manager to kaltura player) - Plugins hold a reference to KP and not PK.
KP doesn't have CorsType. Changed to be taken from core import (reference to KP / PK is not needed)
  • Loading branch information
RoyBregman authored Aug 9, 2020
1 parent 2cc0780 commit 3ef8a39
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/vr.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import {StereoEffect} from './stereo-effect';
import {Error} from './errors';
import './style.css';

const {Error: PKError, FakeEvent, Utils} = core;
const {Error: PKError, FakeEvent, Utils, CorsType} = core;
/**
* The video tag class.
* @type {string}
Expand Down Expand Up @@ -130,7 +130,7 @@ class Vr extends BasePlugin {
(env.os.name === 'iOS' || env.browser.name === 'Safari' || env.browser.name === 'Android Browser')
) {
this._crossOriginSet = true;
this.player.crossOrigin = this.player.CorsType.ANONYMOUS;
this.player.crossOrigin = CorsType.ANONYMOUS;
}
}

Expand Down

0 comments on commit 3ef8a39

Please sign in to comment.