<AWT Dev> Webrev available for rfe 6929295 Generic support of mouse event transformation for AWT/Swing

Piet Blok pbhome at ziggo.nl
Fri Apr 16 09:08:17 PDT 2010


Hi Anthony,

> Hi Piet,
> 
> On 04/15/2010 03:48 PM, Piet Blok wrote:
>>> src/share/classes/java/awt/Component.java
>>>> 2045                 if (peer instanceof LightweightPeer) {
>>>> 2089                 if (peer instanceof LightweightPeer) {
>>>
>>> I suggest using the isLightweight() method instead.
>> 
>> I followed the exact structure as found in the existing method 
>> getLocationOnScreen_NoTreeLock(). I regarded that existing code more or 
>> less as "sacred" and to be followed.
> 
> Still, I suggest to replace these with calls to the isLightweight() 
> method since that is considered to be "the right" way to check whether a 
> component is lightweight.
> 
> Otherwise I'm OK with the AWT part of the fix. Great work. Thanks!

Done. The webrev is refreshed.

Thanks for reviewing.
Piet

> 
>>>> http://www.pbjar.org/OpenJDK/6929295/webrev/
>




More information about the awt-dev mailing list