On 11/21/2014 5:41 PM, Alexey Ivanov wrote:
Hi Alexandr,

Although I'm not a reviewer, the fix look good to me.

The fix works with JColorChooserDnDTest and does not cause ClassCastException in the test for 8024061. Yet the latter test, test/sun/awt/dnd/8024061/bug8024061.java, fails. I propose to remove the additional check that data is null, lines 280-282, from the test.

    Thank you for the review.
    I have updated the test bug8024061:
       http://cr.openjdk.java.net/~alexsch/8065098/webrev.01

   Thanks,
   Alexandr.


Regards,
Alexey

On 21.11.2014 13:38, Alexandr Scherbatiy wrote:

Hello,

Could you review the fix:
  bug: https://bugs.openjdk.java.net/browse/JDK-8065098
  webrev: http://cr.openjdk.java.net/~alexsch/8065098/webrev.00

The issue appears after the fix 8024061 Exception thrown when drag and drop between two components is executed quickly. In some way the mime type of the dropped color has javaJVMLocalObjectMimeType on the second JVM.

The issue should be fixed in JDK 8 as the regression so the easy safe fix that prevents ClassCastException is added. The full fix that includes a proper synchronization between XAWT and EDT threads on Linux can be added later.

Thanks,
Alexandr.



Reply via email to