| Summary: | [GStreamer] TextCombiner has unlinked internal encoders | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Philippe Normand <pnormand> | ||||||
| Component: | Platform | Assignee: | Philippe Normand <pnormand> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | calvaris, cgarcia, ews-watchlist, gustavo, menard, vjaquez, webkit-bug-importer | ||||||
| Priority: | P2 | Keywords: | InRadar | ||||||
| Version: | WebKit Nightly Build | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| Attachments: |
|
||||||||
Created attachment 432191 [details]
Patch
Comment on attachment 432191 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=432191&action=review > Source/WebCore/platform/graphics/gstreamer/TextCombinerPadGStreamer.cpp:39 > + bool processStickyEvents { true }; shouldProcessStickyEvents Committed r279285 (239164@main): <https://commits.webkit.org/239164@main> |
Created attachment 432187 [details] pipeline dump Oddly this doesn't seem to introduce test failures, unless playbin3 is enabled. In some tests such as media/track/in-band/track-in-band-kate-ogg-cues-added-once.html the combiner pads can receive multiple caps events describing the same stream, that's not good because then new vtt encoders are added to the combiner and remain unlinked. See attached pipeline dump.