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

[Prem Balakrishnan]

Hi Semyon,

 

Yup tried with Window:

Dialog checks parent's type internally, if the type is Window following exception is thrown.

Exception in thread "main" java.lang.IllegalArgumentException: Wrong parent window

 

Code snippet  from awt/Dialog.java 

 public Dialog(Window owner, String title, ModalityType modalityType) {

        super(owner);

       if ((owner != null) &&

            !(owner instanceof Frame) &&

            !(owner instanceof Dialog))

        {

            throw new IllegalArgumentException("Wrong parent window");

        }

...

...

 

Regards,
Prem

 

From: Semyon Sadetsky 
Sent: Wednesday, January 13, 2016 3:40 PM
To: Prem Balakrishnan; Ambarish Rapte; Sergey Bylokhov; awt-dev at openjdk.java.net
Subject: Re: Review Request for 8057574: Inconsistent behavior for setBackground (Windows/Linux)

 

Hi,

On 1/13/2016 12:20 PM, Prem Balakrishnan wrote:

Hi Semyon,

 

1.       Patch is updated with the relative test for the fix. 

Updated Webrev: http://cr.openjdk.java.net/~arapte/prem/8057574/webrev.01/

Bug: https://bugs.openjdk.java.net/browse/JDK-8057574 

 

2.       The issue is observed when Frame is parent and Dialog is Child. (Scenario reported by Alexander Stepanov)

The above issue is resolved with the same fix.  

Did you try Window child?

--Semyon



Test related to this scenario is also updated with the patch.

 

Regards,

Prem

 

 

 

 

From: Semyon Sadetsky 
Sent: Monday, January 11, 2016 6:00 PM
To: Prem Balakrishnan; Ambarish Rapte; HYPERLINK "mailto:awt-dev at openjdk.java.net"awt-dev at openjdk.java.net
Subject: Re: Review Request for 8057574: Inconsistent behavior for setBackground (Windows/Linux)

 

On 1/11/2016 2:18 PM, Prem Balakrishnan wrote:




Hi Semyon,

 

Thanks for the review comments.

 

1.       I will add a relative test for the fix.

 

2.       The same issue is observed with Frame also,

        As the original issue is reported for dialog, fix submitted only for dialog. 

Should I combine  fix for  Dialog and Frame in the same patch?

Yes. I would consider it as the same issue. All window types may have this issue.

--Semyon




 

Regards,

Prem

 

 

From: Semyon Sadetsky 
Sent: Monday, January 11, 2016 3:32 PM
To: Prem Balakrishnan; Ambarish Rapte; HYPERLINK "mailto:awt-dev at openjdk.java.net"awt-dev at openjdk.java.net
Subject: Re: Review Request for 8057574: Inconsistent behavior for setBackground (Windows/Linux)

 

Hi Prem,

By default a fix requires a regression test which fails before the fix an passes after. When it is absent noreg-??? label should be set to the JIRA ticket. See http://openjdk.java.net/guide/changePlanning.html for possible noreg suffixes.
But it seems to me that reg test can be written for the issue.

In JIRA Alexander Stepanov noted that the issue takes place for Frame as well but your solution is fixing the Dialog only. Did you run the scenario with Frame?

--Semyon

On 1/4/2016 11:28 AM, Prem Balakrishnan wrote:

Hi,

Please review fix for JDK9,

 

Bug: https://bugs.openjdk.java.net/browse/JDK-8057574 

Webrev: HYPERLINK "http://cr.openjdk.java.net/%7Earapte/prem/8057574/webrev.00/"http://cr.openjdk.java.net/~arapte/prem/8057574/webrev.00/

 

Issue:

It is Platform specific Issue - LINUX

If Child Dialog's Background color is NOT set explicitly, Child Dialog inherits Parent Dialog's Background Color.

(In Windows OS Child Dialog is NOT inheriting parent Dialog's Background Color)

 

Cause:

Default Background color for Dialog is not initialized.

 

Fix:

If Background color is not set explicitly , default Background color is set at the time of Dialog's initialization.

 

Thanks,

Prem

 

 

 

 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20160113/1e262880/attachment-0001.html>