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

Cherry pick PR #3159: Introduce new logic for HTMLMediaElement.buffered #3184

Merged
merged 2 commits into from
May 8, 2024

Conversation

cobalt-github-releaser-bot
Copy link
Collaborator

Refer to the original PR: #3159

This new logic is aligned with upstream Chromium, and with the HTML spec:

Because this is a potentially disruptive change, the code is gated behind a new H5VCC flag. Use
"MediaElement.EnableUsingMediaSourceBufferedRange" to enable this change.

b/338452286

Change-Id: I52052b2fc5a2c0a1a1c92552434cf53f3bf407e2

This new logic is aligned with upstream Chromium, and with the [HTML
spec](https://html.spec.whatwg.org/#dom-media-buffered):

*
[HTMLMediaElement::BufferedInternal](https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/html/media/html_media_element.cc;l=3769-3772;drc=eb7475582501e654b3c0bb94be9fdbadfd6b83a0;bpv=0;bpt=0)
*
[MediaSource::SeekableInternal](https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/modules/mediasource/media_source.cc;l=882-883;drc=d911cdd7a76cdf8895915eae6b88e3bff02a3072;bpv=1;bpt=0)

Because this is a potentially disruptive change, the code is gated
behind a new H5VCC flag. Use
`"MediaElement.EnableUsingMediaSourceBufferedRange"` to enable this
change.

b/338452286

Change-Id: I52052b2fc5a2c0a1a1c92552434cf53f3bf407e2
(cherry picked from commit a348ae2)
@at-ninja at-ninja enabled auto-merge (squash) May 7, 2024 18:37
….lts.1+-3159

Change-Id: I300daec00c8ff25c72fb86a0d11746a8cb339250
@at-ninja at-ninja merged commit afc309e into 25.lts.1+ May 8, 2024
295 of 297 checks passed
@at-ninja at-ninja deleted the cherry-pick-25.lts.1+-3159 branch May 8, 2024 17:12
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.

2 participants