qt: x11: do not call `::{create,destroy}Fbo()` before making the context current and relax assertions

Rather than having an assertion regarding the context became current, we can simply early return in these functions. If the context could not be made current, it is likely that we are in some sort of transitional phase and resizeFbo() would be called later when the state is stable either directly or indirectly.

render(), on the other is periodically called anyway. It should be fine to simply return early instead of triggering assertion failure. When the context can be made current, it can do its job.

Request review @chub.

added MRStatus::Reviewable label

changed milestone to %4.0

added Component::Interface: Qt label

requested review from @chub

added MRStatus::WaitingForReviewerFeedback label and removed MRStatus::Reviewable label

approved this merge request

added MRStatus::Acceptable label and removed MRStatus::WaitingForReviewerFeedback label

added MRStatus::Accepted label and removed MRStatus::Acceptable label

MR Acceptance result

This MergeRequest has been Accepted! Congratulations.

MR acceptance checks details:

• MR should be considered mergeable by Gitlab
• Last pipeline should be successful
• MergeRequest should have at least one external review and/or vote
• All threads should be resolved, have votes and score > 0
• MergeRequest should have no activity (threads/votes) for (72h/72h)

This message was automatically generated by homer-bot.

added 61 commits

• a2d48935...d1a0dc70 - 59 commits from branch videolan:master
• 3257b2e4 - qt: relax made current assertions in `CompositorX11UISurface`
• 99162581 - qt: make context current before calling `::createFbo()` and `::destroyFbo()`...

Compare with previous version

enabled an automatic merge when all merge checks for 99162581 pass

merged