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

[Alexander Zuev]

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

http://cr.openjdk.java.net/~kizune/7177144/webrev.03

WBR,
Alex

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