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

[Sergey Bylokhov]

10.08.2012 16:26, 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.
At least one of them can be moved from coalesceMouseEvent to coalesceEvent

         if (e instanceof MouseEvent && !(e instanceof 
SunDropTargetEvent)) {

         }

in cacheEQItem() it executes almost always. Why we should not cache 
these events?

>
> 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


-- 
Best regards, Sergey.