On 11/19/21 1:46 PM, Rémi Bernon wrote:
On 11/19/21 13:17, Jacek Caban wrote:
Hi Rémi,
On 11/19/21 12:23 PM, Rémi Bernon wrote:
This makes register_builtin_classes to not be called anymore in the desktop process, and IME window creation to fail when the systray window is created.
Thanks for looking at it. I was planning to send the attached patch after a bit more testing, I'd expect it to solve the problem that you noticed.
Jacek
Yes, looks like it works as well, but I'm a bit afraid that doing the builtin class registration in GetDesktopWindow is a bet that it'll be called by the desktop process and called soon enough, which I'm not sure we can have any guarantee about.
I sent a bit different version of the patch that adds another call that will make sure that it's called soon enough in explorer process and removed the call from load_driver() entirely. Note that there are other, not desktop related, cases that we should handle as well. For example wineandroid.drv is also a sounds driver, so it may be loaded without user32/win32u involvement.
Thanks,
Jacek