https://bugs.winehq.org/show_bug.cgi?id=48732
--- Comment #16 from Zebediah Figura z.figura12@gmail.com --- (In reply to Gabriel Ivăncescu from comment #15)
First, it's obviously more correct in terms of what Windows does; even if it's an implementation detail, it looks some applications depend on it, so we should IMO implement it as close as possible.
Clearly the application doesn't depend on this exact code (if it's not using a hook or subclassing or anything), so it must depend on some distant side effect. The only question is whether that side effect can be achieved in a more reasonable manner, which is why I really want to know what it is.
Secondly, when closing Media Player Classic while playing a video, there's a short flashing/flicker window on the top-left of the screen, which appears because of SetParent, which also seems to cause some fast focus changes. This patch fixes that as well, and so is a smoother experience overall.
Is it really correct that SetParent() on a non-visible window should cause any focus changes? That seems surprising to me. If it's not correct, I think that's what should be fixed instead.