<AWT Dev> [8] Please review fix for 7177144: [macosx] Drag and drop not working (regression in 7u6)

Alexander Zuev alexander.zuev at oracle.com
Mon Aug 13 07:28:12 PDT 2012

Well, it works this way too and fix is much more compact so here it is - 
a new version available at



On 8/10/12 19:41, Artem Ananiev wrote:
> 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

More information about the awt-dev mailing list