<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Hello.<div><br></div><div>This is a reminder. Could I please get the second review on this?</div><div><br></div><div>The bug:&nbsp;<a href="https://bugs.openjdk.java.net/browse/JDK-8024163">https://bugs.openjdk.java.net/browse/JDK-8024163</a></div><div>The fix:&nbsp;<a href="http://cr.openjdk.java.net/~pchelko/8024163/webrev.01">http://cr.openjdk.java.net/~pchelko/8024163/webrev.01</a></div><div><br></div><div>Thank you. With best regards. Petr.</div><div><br></div><div><div><div>On Sep 25, 2013, at 3:05 PM, Petr Pchelko &lt;<a href="mailto:petr.pchelko@oracle.com">petr.pchelko@oracle.com</a>&gt; wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Hello, AWT Team.<br><br>Please review the updated version of this fix. <br>It's available at: <a href="http://cr.openjdk.java.net/~pchelko/8024163/webrev.01/">http://cr.openjdk.java.net/~pchelko/8024163/webrev.01/</a><br><br>I have changed the dragExit events generation a bit and added a couple of tests.<br><br>With best regards. Petr.<br><br>On Sep 24, 2013, at 6:30 PM, Petr Pchelko &lt;<a href="mailto:petr.pchelko@oracle.com">petr.pchelko@oracle.com</a>&gt; wrote:<br><br><blockquote type="cite">Hello, AWT Team.<br><br>Please review the fix for the following issue:<br><a href="https://bugs.openjdk.java.net/browse/JDK-8024163">https://bugs.openjdk.java.net/browse/JDK-8024163</a><br>The fix is available here:<br>http://cr.openjdk.java.net/~pchelko/8024163/webrev.00/<br><br>The problem is with the DropTarget Enter/Exit events. For real heavyweights they are generated by native code. For lightweights - in shared code. But for AWT components they should be generated in CDropTargetContextPeer.<br>Before the fix these events could be generated incorrectly: sometimes duplicated events were sent (this broke autoscrolling) and sometimes events were not sent at all - this caused NPEs in the shared code.<br>The insideTarget boolean was replaced by a reference to DropTarget to handle nested components correctly.<br><br>Tested on Mac OS X (no shared code affected). <br>No new regression test failures.<br><br>With best regards. Petr.<br><br><br></blockquote><br></blockquote></div><br></div></body></html>