Crash Deleting Windows since addition of isWindowOnCurrentVirtualDesktop

It’s still possible to get a crash if isWindowOnCurrentVirtualDesktop happens to dispatch a dpi change, which resets the shadower -that’s what I tried to fix with the WeakReference. It needs a lot of bad luck though.

This seems to fix the repaint issue we were seeing. Our splash screen now shows correctly and the crash we were getting showing alert windows is fixed.

This is probably good enough for us to ship a beta to testers with for now but the crash @kamedin points out is a little worrying. The fact that any messages are dispatched synchronously feels a bit wrong. Could they just be caught and reposted?

Just to confirm, IsWindowOnCurrentVirtualDesktop happens to be calling PeekMessage, god knows why.

This should be fixed on develop now:

1 Like