Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[application] Fix black screen when switching from PVR Live TV channel to PVR Radio channel. #17574

Merged
merged 2 commits into from
Apr 4, 2020

Conversation

ksooo
Copy link
Member

@ksooo ksooo commented Mar 28, 2020

This fixes #17556

When switching from a PVR Live TV to a PVR Radio channel, active window must be changed from WINDOW_FULLSCREEN_VIDEO to WINDOW_VISUALISATION. On stream change VideoPlayer fires a TMSG_SWITCHTOFULLSCREEN event which is handled by CApplication::OnApplicationMessage. The check for the active window there prevents switching the window, because when the event arrives the active window is still WINDOW_FULLSCREEN_VIDEO.

Same problem occurs when switching from Radio to Live TV since #17225, btw.

Removing the checks for the active window in CApplication::OnApplicationMessage fixes the issue and has no other negative side effects.

This issue is spacial to PVR, because VideoPalyer has special handling for PVR stream changes (fast channel switch).

Runtime tested on macOS and Android, latest Kodi master.

Hard to find a reviewer for this I guess (who is maintainer of CApplication?) , but maybe @phunkyfish can look at the change.

@ksooo ksooo added Type: Fix non-breaking change which fixes an issue Component: PVR v19 Matrix labels Mar 28, 2020
@ksooo ksooo added this to the Matrix 19.0-alpha 1 milestone Mar 28, 2020
@phunkyfish
Copy link
Contributor

How would we know this doesn’t impact anything outside of PVR?

Would it still work if only the visualisation check was omitted?

@ksooo
Copy link
Member Author

ksooo commented Mar 28, 2020

How would we know this doesn’t impact anything outside of PVR?

I did extensive runtime tests with all kind of media. If I missed something, our general Kodi testing strategy will come to play - let our (alpha-)users test it in the nightlies.

... and @MilhouseVH could you add this PR to your builds, please?

@ksooo
Copy link
Member Author

ksooo commented Mar 28, 2020

Would it still work if only the visualisation check was omitted?

No. At least vis and fullscreen video checks must go. Otherwise either TV->Radio or Radio->TV will not work.

@MilhouseVH
Copy link
Contributor

... and @MilhouseVH could you add this PR to your builds, please?

Will do - should be in #0329

@ksooo
Copy link
Member Author

ksooo commented Apr 4, 2020

No negative feedback from Milhouse build testers. Going to merge this now.

@ksooo ksooo merged commit 6de683c into xbmc:master Apr 4, 2020
@ksooo ksooo deleted the fix-issue17556 branch April 4, 2020 07:36
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Apr 6, 2020
[application] Fix black screen when switching from PVR Live TV channel to PVR Radio channel.
Maven85 pushed a commit to Maven85/kodi that referenced this pull request May 5, 2020
[application] Fix black screen when switching from PVR Live TV channel to PVR Radio channel.
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Aug 3, 2020
[application] Fix black screen when switching from PVR Live TV channel to PVR Radio channel.
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Aug 4, 2020
[application] Fix black screen when switching from PVR Live TV channel to PVR Radio channel.
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Aug 4, 2020
[application] Fix black screen when switching from PVR Live TV channel to PVR Radio channel.
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Aug 5, 2020
[application] Fix black screen when switching from PVR Live TV channel to PVR Radio channel.
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Aug 6, 2020
[application] Fix black screen when switching from PVR Live TV channel to PVR Radio channel.
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Aug 6, 2020
[application] Fix black screen when switching from PVR Live TV channel to PVR Radio channel.
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Aug 6, 2020
[application] Fix black screen when switching from PVR Live TV channel to PVR Radio channel.
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Aug 7, 2020
[application] Fix black screen when switching from PVR Live TV channel to PVR Radio channel.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: PVR Type: Fix non-breaking change which fixes an issue v19 Matrix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Black screen when switching from PVR TV channel to PVR radio channel
3 participants