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

[Artem Ananiev]

Hi, Alex,

I like this version of the fix much more than the previous one :) Could 
you also add a short comment to the code, why we handle DnD events this 
way, please?

Thanks,

Artem

On 8/13/2012 6:28 PM, Alexander Zuev wrote:
> 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
>