Hi Alexander,

I guess I'm a bit late with the review, but my comment also concerns the fix for 7u6. You're changing the shared file SunDropTargetContextPeer.java and reverting a boolean argument when calling postDropTargetEvent. Can this change affect other platforms? Have you run some tests for DnD area on Win and X11 to verify nothing is broken?

--
best regards,
Anthony

On 5/24/2012 1:36 PM, Alexander Zuev wrote:
Hello,

  please review my fix for the
7171163: [macosx] Shortcomings in the design of the secondary native event loop made JavaFX DnD deadlock

Bug can be found at http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7171163

  Webrev can be found at:
http://cr.openjdk.java.net/~kizune/7171163/webrev.01/

With best regards,
Alexander Zuev

Reply via email to