<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<div class="moz-cite-prefix">On 11/19/21 1:46 PM, Rémi Bernon wrote:<br>
</div>
<blockquote type="cite"
cite="mid:d28a025f-c600-a4bd-4cfa-b8701f97041f@codeweavers.com">On
11/19/21 13:17, Jacek Caban wrote:
<br>
<blockquote type="cite" style="color: #007cff;">Hi Rémi,
<br>
<br>
On 11/19/21 12:23 PM, Rémi Bernon wrote:
<br>
<blockquote type="cite" style="color: #007cff;">This makes
register_builtin_classes to not be called anymore in the
<br>
desktop process, and IME window creation to fail when the
systray window
<br>
is created.
<br>
</blockquote>
<br>
<br>
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.
<br>
<br>
<br>
Jacek
<br>
</blockquote>
<br>
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.
</blockquote>
<p><br>
</p>
<p>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.<br>
</p>
<p><br>
</p>
<p>Thanks,</p>
<p>Jacek<br>
</p>
</body>
</html>