diff --git a/docs/tutorials/01-settings-and-overrides.md b/docs/tutorials/01-settings-and-overrides.md index f61c33ce..8d560f55 100644 --- a/docs/tutorials/01-settings-and-overrides.md +++ b/docs/tutorials/01-settings-and-overrides.md @@ -17,7 +17,6 @@ In order to add an override, simply add an override object to the `window.bigscr | `restartTimeout` | This is used after a seek attempted to ensure that a seek has happened correctly. | number | | `disableMediaSourceUnload` | When tearing down in HTML5 we unload the source as part of the html5 spec. This prevents this from happening as it was causing issues on devices. | boolean | | `legacySubtitles` | This can be used to render subtitles using our legacy method, rather than the new method which utilises the third party imscJS library. | boolean | -| `liveUhdDisableSentinels` | Disables any sentinels when consuming live UHD content | boolean | | `cacheSeekableRange` | Caches the seekable range so it can't be requested more than every 250ms for devices that struggle. | boolean | | `resetMSEPlayer` | Forces the `dash.js` MediaPlayer and HTML MediaElement to be fully destroyed/removed when `reset()` is called on the MSE Strategy. Reset is called from `playerComponent.js` on a CDN failover, and usually does nothing (allowing a new source to be attached to the current Media Player). | boolean | diff --git a/src/playbackstrategy/legacyplayeradapter.js b/src/playbackstrategy/legacyplayeradapter.js index 61257c89..03bf4640 100644 --- a/src/playbackstrategy/legacyplayeradapter.js +++ b/src/playbackstrategy/legacyplayeradapter.js @@ -9,8 +9,7 @@ function LegacyPlayerAdapter(mediaSources, playbackElement, isUHD, player) { const manifestType = mediaSources.time().manifestType const setSourceOpts = { - disableSentinels: - !!isUHD && manifestType === ManifestType.DYNAMIC && window.bigscreenPlayer?.overrides?.liveUhdDisableSentinels, + disableSentinels: false, disableSeekSentinel: !!window.bigscreenPlayer?.overrides?.disableSeekSentinel, } diff --git a/src/playbackstrategy/legacyplayeradapter.test.js b/src/playbackstrategy/legacyplayeradapter.test.js index a60912ef..0f501bcf 100644 --- a/src/playbackstrategy/legacyplayeradapter.test.js +++ b/src/playbackstrategy/legacyplayeradapter.test.js @@ -239,33 +239,6 @@ describe("Legacy Playback Adapter", () => { } ) - it("should disable all sentinels for a dynamic UHD stream when configured to do so", () => { - window.bigscreenPlayer.overrides = { - liveUhdDisableSentinels: true, - } - - mockMediaSources.time.mockReturnValueOnce({ manifestType: ManifestType.DYNAMIC }) - mockMediaSources.currentSource.mockReturnValueOnce("mock://media.src/") - - const isUHD = true - const mediaPlayer = createMockMediaPlayer() - - const legacyAdapter = LegacyAdapter(mockMediaSources, playbackElement, isUHD, mediaPlayer) - - legacyAdapter.load("video/mp4") - - expect(mediaPlayer.initialiseMedia).toHaveBeenCalledWith( - "video", - "mock://media.src/", - "video/mp4", - playbackElement, - { - disableSeekSentinel: false, - disableSentinels: true, - } - ) - }) - it("should disable seek sentinels if we are configured to do so", () => { window.bigscreenPlayer.overrides = { disableSeekSentinel: true,