<AWT Dev> Review Request for 8057574 : inconsistent behavior for setBackground (Windows/Linux)

[Ambarish Rapte]

Hi Prem,
	The fix look fine to me.

Regards,
Ambarish

-----Original Message-----
From: Prem Balakrishnan 
Sent: Friday, April 29, 2016 10:36 AM
To: Sergey Bylokhov; Ambarish Rapte; Semyon Sadetsky; awt-dev at openjdk.java.net
Subject: RE: Review Request for 8057574 : inconsistent behavior for setBackground (Windows/Linux)

Hi Sergey,

Thank you for the Review.
Update patch as per review comments.
http://cr.openjdk.java.net/~pkbalakr/8057574/webrev.01/ 

Regards,
Prem

-----Original Message-----
From: Sergey Bylokhov
Sent: Thursday, April 28, 2016 6:16 PM
To: Prem Balakrishnan; Semyon Sadetsky; Ambarish Rapte; awt-dev at openjdk.java.net
Subject: Re: Review Request for 8057574 : inconsistent behavior for setBackground (Windows/Linux)

Hi, Prem.
The fix looks fine. But please preserve the comments in XWindowPeer.java

On 25.04.16 9:00, Prem Balakrishnan wrote:
> Hi*,*
>
> Please review fix for JDK9,
>
> *Bug:*https://bugs.openjdk.java.net/browse/JDK-8057574
>
> *Webrev:*http://cr.openjdk.java.net/~pkbalakr/8057574/webrev.00/
>
> *Issue:*
>
> inconsistent behavior for setBackground (Windows/Linux)
>
> Inconsistency also exists for Foreground and Font properties across 
> platforms (Win and Linux)
>
> *Fix: *
>
> Uniform behaviour is maintained by making , child component NOT to 
> inherit  parent properties (Background, Foreground and Font)
>
> across all the platforms(Win/Linux/Mac).
>
> *Regression and JCK tests passed without causing any regression with 
> the suggested fix.*
>
> Regards,
> Prem
>


--
Best regards, Sergey.