RESOLVED FIXED 236453
[GStreamer][MSE] Let MSE set natural video size when using holepunch 
https://bugs.webkit.org/show_bug.cgi?id=236453
Summary [GStreamer][MSE] Let MSE set natural video size when using holepunch
Enrique Ocaña
Reported 2022-02-10 10:35:10 PST
Currently the holepunch implementation completely overrides the video size to a hardcoded value. It would be desirable to let the m_videoSize value set by MediaPlayerPrivateGStreamerMSE::setInitialVideoSize() take precedence over the hardcoded value.
Attachments
Patch (2.94 KB, patch)
2022-02-10 10:43 PST, Enrique Ocaña 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Enrique Ocaña
Comment 1 2022-02-10 10:43:35 PST
Created attachment 451565 [details] Patch
Xabier Rodríguez Calvar
Comment 2 2022-02-10 23:20:02 PST
Comment on attachment 451565 [details] Patch I'm ok, but let's wait for Miguel, our holepunch expert to give the final green light.
Miguel Gomez
Comment 3 2022-02-14 07:03:14 PST
(In reply to Xabier Rodríguez Calvar from comment #2) > Comment on attachment 451565 [details] > Patch > > I'm ok, but let's wait for Miguel, our holepunch expert to give the final > green light. LGTM
EWS
Comment 4 2022-02-15 04:03:12 PST
Committed r289800 (247265@main): <https://commits.webkit.org/247265@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 451565 [details].
Radar WebKit Bug Importer
Comment 5 2022-02-15 04:04:58 PST
<rdar://problem/88957858>
Note You need to log in before you can comment on or make changes to this bug.