Skip to content

Conversation

@asurdej-comcast
Copy link

@asurdej-comcast asurdej-comcast commented Nov 5, 2025

… text limit specified

Ignore missing "Text" limit in MSE_MAX_BUFFER_SIZE env when calculating max buffer size for init segment.
With current impl, missing any of track type A/V/T from MSE_MAX_BUFFER_SIZE causes WebKit to fallback to default value from HTMLMediaElement/settings that is ~15MB for WPE. Providing higher video and audio limits in the env we can still end up in QuotaExceededError because of that.

Assume video and audio limits are enough to calculate accumulative max buffer size for any type of track.
c9318f6

Build-Tests Layout-Tests
✅ 🛠 wpe-246-amd64-build ✅ 🧪 wpe-246-amd64-layout
✅ 🛠 wpe-246-arm32-build ✅ 🧪 wpe-246-arm32-layout

… text limit specified

Ignore missing "Text" limit in MSE_MAX_BUFFER_SIZE env when calculating max buffer size
for init segment.
With current impl, missing any of track type A/V/T from MSE_MAX_BUFFER_SIZE causes WebKit
to fallback to default value from HTMLMediaElement/settings that is ~15MB for WPE.
Providing higher video and audio limits in the env we can still end up in QuotaExceededError
because of that.

Assume video and audio limits are enough to calculate accumulative max buffer size
for any type of track.
@philn philn requested a review from eocanha November 5, 2025 18:25
vivienne-w added a commit to vivienne-w/WebKit that referenced this pull request Nov 6, 2025
… text limit specified

https://bugs.webkit.org/show_bug.cgi?id=302072

Reviewed by NOBODY (OOPS!).

Ignore missing "Text" limit in MSE_MAX_BUFFER_SIZE env when calculating max buffer size
for init segment.
With current impl, missing any of track type A/V/T from MSE_MAX_BUFFER_SIZE causes WebKit
to fallback to default value from HTMLMediaElement/settings that is ~15MB for WPE.
Providing higher video and audio limits in the env we can still end up in QuotaExceededError
because of that.

Assume video and audio limits are enough to calculate accumulative max buffer size
for any type of track.

Original author: Andrzej Surdej <Andrzej_Surdej@comcast.com>
See: WebPlatformForEmbedded/WPEWebKit#1578

* Source/WebCore/platform/graphics/gstreamer/mse/SourceBufferPrivateGStreamer.cpp:
(WebCore::SourceBufferPrivateGStreamer::platformMaximumBufferSize const):
@vivienne-w vivienne-w self-assigned this Nov 6, 2025
@vivienne-w vivienne-w added the upstream Related to an upstream bug (or should be at some point) label Nov 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

upstream Related to an upstream bug (or should be at some point)

Development

Successfully merging this pull request may close these issues.

2 participants