macosx: Refactor setupVoutForWindow in VLCVideoOutputProvider
This PR's main changes regarding setupVoutForWindow
are:
- It moves much of the introspection into
mainInstance.libraryWindow
out ofVLCVideoOutputProvider
and intoVLCLibraryWindow
, where the actions that need doing to setup/unsetup video playback called by the setter ofhasActiveVideo
(which we always call regardless of the video window type) - It moves the creation of the borderless window (video wallpaper / no decoration window) into its own function, reducing the complexity of
setupVoutForWindow
- Cleans up redundant conditional cases and unnecessary assignments
Signed-off-by: Claudio Cambra claudio.cambra@gmail.com
Edited by Claudio Cambra
Merge request reports
Activity
Filter activity
added 1 commit
- b1998062 - macosx: Refactor setupVoutForWindow in VLCVideoOutputProvider
added MRStatus::Reviewable label
changed milestone to %4.0
added Component::Interface: macOS Platform::macOS labels
added MRStatus::Acceptable label and removed MRStatus::Reviewable label
added 60 commits
-
b1998062...6dea7695 - 59 commits from branch
videolan:master
- 3bf34cfb - macosx: Refactor setupVoutForWindow in VLCVideoOutputProvider
-
b1998062...6dea7695 - 59 commits from branch
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)
-
added 4 commits
-
3bf34cfb...cd238cef - 3 commits from branch
videolan:master
- 1eb10010 - macosx: Refactor setupVoutForWindow in VLCVideoOutputProvider
-
3bf34cfb...cd238cef - 3 commits from branch
Please register or sign in to reply