Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: changeType() is Not Fully Implemented on Sony PS5 #4149

Merged
merged 4 commits into from Mar 29, 2023

Conversation

agajassi
Copy link
Contributor

Fixes issue/4148 by offering client an option to use useChangeTypeForTrackSwitch flag to avoid errors involving changeType() not being supported on certain platforms.

Check useChangeTypeForTrackSwitch flag first
@dsilhavy dsilhavy added this to the 4.7.0 milestone Mar 22, 2023
@dsilhavy dsilhavy self-requested a review March 22, 2023 06:54
Copy link
Contributor

@vodlogic vodlogic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The StreamController seems to just rely on whether the browser supports the changeType property instead of using this configuration. Perhaps we should add this config check there too to ensure consistency thoughout the dash.js code? I think just because the browser supports the changeType function, does not necessarily mean it works properly with the video pipeline of the device.

if (keys.length > 0 && sinks[keys[0]].getBuffer().changeType) {

adding this config check there too to ensure consistency throughout the dash.js code
@agajassi
Copy link
Contributor Author

agajassi commented Mar 23, 2023

@vodlogic Yes agree. Added config check in StreamController as well.

@dsilhavy
Copy link
Collaborator

@agajassi @vodlogic : I think we should move the settings check to _canSourceBuffersBeReused.

supportsChangeType && settings.get().streaming.buffer.useChangeTypeForTrackSwitch

That way we clearly separate settings from capabilities. We also still apply the changeType transition if the user/app changes the setting after _activateStream.

@dsilhavy
Copy link
Collaborator

@agajassi Did you find the time to check my comment yet?

@agajassi
Copy link
Contributor Author

@agajassi Did you find the time to check my comment yet?
Sorry, I got a little busy over the weekend. Yes I saw your comment. I will give it a try and run some tests today. Thanks for the suggestion.

@agajassi
Copy link
Contributor Author

I moved settings check from _activateStream to _canSourceBuffersBeReused

@agajassi agajassi requested review from vodlogic and removed request for dsilhavy March 28, 2023 19:45
@dsilhavy dsilhavy merged commit 389c145 into Dash-Industry-Forum:development Mar 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

changeType() is Not Fully Implemented on Sony PS5
3 participants