Bug 282749 – [GStreamer] Video dimensions are wrong since GStreamer 1.24.9

• Component: WPE WebKit
• Status: RESOLVED FIXED

[Theo]

Hello,

This commit from GStreamer 1.24 branch 7e438dfaa7cc6877e70d139451dfcc5e4e6b4756, makes WPEWebkit drop the stream-collection because it comes from decodebin3 now and not webkit source element anymore

As a result, updateTracks is never called and the video dimensions never updated

You can reproduce the issue using MSE, here is a test page https://nickdesaulniers.github.io/netfix/demo/bufferAll.html

Thanks

[Theo]

Created attachment 473164 [details]
screenshot of the video dimension bug

[Theo]

Created attachment 473165 [details]
screenshot of the expected video dimension

[Philippe Normand]

So you have a "Ignoring redundant STREAM_COLLECTION" message in the logs?

[Philippe Normand]

Oh no the message is dropped by uridecodebin3. Well, iiuc making the MSE source answer selectable queries positively would solve the issue.

The mediastreamsrc element already does that, fwiw.

[Theo]

(In reply to Philippe Normand from comment #3)
> So you have a "Ignoring redundant STREAM_COLLECTION" message in the logs?

That's right because the message comes from decodebin3

[Theo]

(In reply to Philippe Normand from comment #4)
> Oh no the message is dropped by uridecodebin3. Well, iiuc making the MSE
> source answer selectable queries positively would solve the issue.
> 
> The mediastreamsrc element already does that, fwiw.

Yes it would fix the issue, I was just not sure of the meaning induced by returning true to this query (there is not much info in the documentation/code)

[Philippe Normand]

Can you send a PR?

[Theo]

(In reply to Philippe Normand from comment #7)
> Can you send a PR?

Yes I will do that

[EWS]

Committed 286347@main (9e9ea966373d): <https://commits.webkit.org/286347@main>

Reviewed commits have been landed. Closing PR #36390 and removing active labels.

[Philippe Normand]

*** Bug 283077 has been marked as a duplicate of this bug. ***