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: Only request initialization segment when it's necessary #6929

Merged
merged 1 commit into from
Jun 28, 2024

Conversation

avelad
Copy link
Member

@avelad avelad commented Jun 27, 2024

According to https://w3c.github.io/media-source/#dom-sourcebuffer-remove

Deleting a range from the source buffer does not delete the initialization segment, so this change will prevent the initialization segment from being requested when it is not needed.

@avelad
Copy link
Member Author

avelad commented Jun 27, 2024

@shaka-bot test

@avelad avelad added type: bug Something isn't working correctly priority: P1 Big impact or workaround impractical; resolve before feature release labels Jun 27, 2024
@avelad avelad added this to the v4.10 milestone Jun 27, 2024
@shaka-bot
Copy link
Collaborator

@avelad: Lab tests started with arguments:

  • pr=6929

@avelad avelad force-pushed the seek-init-segment branch from ca4a0a7 to 68ee757 Compare June 27, 2024 13:20
@shaka-bot
Copy link
Collaborator

Incremental code coverage: No instrumented code was changed.

@avelad avelad requested a review from theodab June 27, 2024 14:17
@avelad avelad merged commit 2f762ec into shaka-project:main Jun 28, 2024
29 of 31 checks passed
@avelad avelad deleted the seek-init-segment branch June 28, 2024 07:15
avelad added a commit that referenced this pull request Jul 1, 2024
According to https://w3c.github.io/media-source/#dom-sourcebuffer-remove

Deleting a range from the source buffer does not delete the
initialization segment, so this change will prevent the initialization
segment from being requested when it is not needed.
avelad added a commit that referenced this pull request Jul 1, 2024
According to https://w3c.github.io/media-source/#dom-sourcebuffer-remove

Deleting a range from the source buffer does not delete the
initialization segment, so this change will prevent the initialization
segment from being requested when it is not needed.
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Aug 27, 2024
@shaka-project shaka-project locked as resolved and limited conversation to collaborators Aug 27, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
priority: P1 Big impact or workaround impractical; resolve before feature release status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants