<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<br>
The fix looks good to me.<br>
<br>
Thanks,<br>
Alexandr.<br>
<br>
<div class="moz-cite-prefix">On 4/18/2016 8:39 AM, Semyon Sadetsky
wrote:<br>
</div>
<blockquote cite="mid:5714FFA9.6000304@oracle.com" type="cite">
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
<br>
<br>
<div class="moz-cite-prefix">On 4/18/2016 6:12 PM, Alexander
Scherbatiy wrote:<br>
</div>
<blockquote cite="mid:5714F973.3070909@oracle.com" type="cite">On
4/1/2016 1:29 PM, Semyon Sadetsky wrote: <br>
<blockquote type="cite">Please review the updated version : <a
moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Essadetsky/8147842/webrev.01/"><a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~ssadetsky/8147842/webrev.01/">http://cr.openjdk.java.net/~ssadetsky/8147842/webrev.01/</a></a>
<br>
<br>
fix was changed to avoid possible recursion in native IME
evens handling which the SendMessage() call might cause. <br>
</blockquote>
<br>
The SendMessage call has been added by the fix JDK-8079595
Resizing dialog which is JWindow parent makes JVM crash <br>
Could you give more details why removing this code back will
not cause new JVM crash? <br>
</blockquote>
I did not remove the code back. Before JDK-8079595 the
DefSubclassProc function was called actually (see
<meta http-equiv="content-type" content="text/html; charset=utf-8">
ComCtl32Util::DefWindowProc). This function is not threadsafe and
that was the root cause of the JDK-8079595.<br>
<br>
--Semyon<br>
<blockquote cite="mid:5714F973.3070909@oracle.com" type="cite"> <br>
Thanks, <br>
Alexandr. <br>
<br>
<br>
<blockquote type="cite"> <br>
--Semyon <br>
<br>
On 3/31/2016 7:38 PM, Semyon Sadetsky wrote: <br>
<blockquote type="cite"> <br>
<br>
On 3/31/2016 7:11 PM, Semyon Sadetsky wrote: <br>
<blockquote type="cite">The solution is to support
positioning for the IME candidate window. <br>
</blockquote>
<br>
It should be : "the IME *composition* window" of cause... <br>
<br>
--Semyon <br>
<br>
</blockquote>
<br>
</blockquote>
<br>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>