Hi, Alex,

what about putting the instanceof check to eventToCacheIndex(), under MOUSE_DRAGGED switch, and return -1 for dnd events? We don't cache such unknown events, as well as don't coalesce them.

Thanks,

Artem

On 8/10/2012 4:26 PM, Alexander Zuev wrote:
Hello,

please review my fix for CR 7177144: [macosx] Drag and drop not working
(regression in 7u6)

I still tink we shouldn't move the instanceof check to the earlier stage
since this check may be costly
so it should not be performed unless other possibilities are ruled out
by the less expensive checks.

Bug description is:
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7177144
Webrev can be found here:
http://cr.openjdk.java.net/~kizune/7177144/webrev.01

With best regards,
Alex

Reply via email to