<AWT Dev> [8] Review request for 8007155 [macosx] Disabled panel takes mouse input in JLayeredPane

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Wed Sep 18 09:01:38 PDT 2013


Hi, Alexander.
Method in the GlobalCursorManager can be private or can be removed:

  137     protected Component findComponentAt(Container con, int x, int y){
  138         return AWTAccessor.getContainerAccessor().
  139                 findComponentAt(con, x, y, false);
  140     }

Can you add @Override to the new method in the Container.java?
Also please add copyright to the html file and update dates in the headers.

On 18.09.2013 19:18, Alexander Scherbatiy wrote:
>
> Hello,
>
> Could you review the fix:
>   bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8007155
>   webrev: http://cr.openjdk.java.net/~alexsch/8007155/webrev.00
>
>
>   The findComponent method in the LWCursorManager class searches from 
> a component under the mouse
>   to the parent components and does not take in the account another 
> components under mouse that intersects the disabled one.
>
>   The fix adds the necessary findComponentAt method to the container 
> accessor and uses it in the *CursorManager classes.
>   The same issues is fixed for the XGlobalCursorManager.
>   Unnecessary call from native method is removed for the 
> WGlobalCursorManager.
>
>   Manual test is added because there is no way to know the actual 
> mouse cursor type.
>
> Thanks,
> Alexandr.
>


-- 
Best regards, Sergey.



More information about the awt-dev mailing list