Re: Swing Dev>[9] Review Request JDK-8163160 [PIT][TEST_BUG] Some issues in java/awt/image/multiresolution/MultiResolutionIcon/IconTest.java

2016-08-16 Thread Rajeev Chamyal
Hello Sergey,

Thanks for the review. In the bug its reported that border of button is grey. 
Instead of button border icon border should be checked. I have updated test 
instructions for this.

Regards,
Rajeev Chamyal

-Original Message-
From: Sergey Bylokhov 
Sent: 16 August 2016 18:28
To: Rajeev Chamyal; Alexander Scherbatiy; swing-dev@openjdk.java.net
Subject: Re: Swing Dev>[9] Review Request JDK-8163160 [PIT][TEST_BUG] Some 
issues in java/awt/image/multiresolution/MultiResolutionIcon/IconTest.java

On 16.08.16 15:57, Sergey Bylokhov wrote:
> The changes looks fine, there are a notice, is it a known issues?

there are a notice about different background of the buttons.

>
> On 16.08.16 15:01, Rajeev Chamyal wrote:
>> Hello All,
>>
>>
>>
>> Please review the following webrev.
>>
>>
>>
>> Webrev: http://cr.openjdk.java.net/~rchamyal/8163160/webrev.00/
>>
>> Bug : https://bugs.openjdk.java.net/browse/JDK-8163160
>>
>> Issue : manual tag was missing in test.
>>
>>
>>
>> Regards,
>>
>> Rajeev Chamyal
>>
>>
>>
>
>


-- 
Best regards, Sergey.


Re: Swing Dev>[9] Review Request JDK-8163160 [PIT][TEST_BUG] Some issues in java/awt/image/multiresolution/MultiResolutionIcon/IconTest.java

2016-08-16 Thread Sergey Bylokhov

On 16.08.16 15:57, Sergey Bylokhov wrote:

The changes looks fine, there are a notice, is it a known issues?


there are a notice about different background of the buttons.



On 16.08.16 15:01, Rajeev Chamyal wrote:

Hello All,



Please review the following webrev.



Webrev: http://cr.openjdk.java.net/~rchamyal/8163160/webrev.00/

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

Issue : manual tag was missing in test.



Regards,

Rajeev Chamyal









--
Best regards, Sergey.


Re: Swing Dev>[9] Review Request JDK-8163160 [PIT][TEST_BUG] Some issues in java/awt/image/multiresolution/MultiResolutionIcon/IconTest.java

2016-08-16 Thread Sergey Bylokhov

The changes looks fine, there are a notice, is it a known issues?

On 16.08.16 15:01, Rajeev Chamyal wrote:

Hello All,



Please review the following webrev.



Webrev: http://cr.openjdk.java.net/~rchamyal/8163160/webrev.00/

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

Issue : manual tag was missing in test.



Regards,

Rajeev Chamyal






--
Best regards, Sergey.


Swing Dev>[9] Review Request JDK-8163160 [PIT][TEST_BUG] Some issues in java/awt/image/multiresolution/MultiResolutionIcon/IconTest.java

2016-08-16 Thread Rajeev Chamyal
Hello All,

 

Please review the following webrev.

 

Webrev: http://cr.openjdk.java.net/~rchamyal/8163160/webrev.00/ 

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

Issue : manual tag was missing in test.

 

Regards,

Rajeev Chamyal

 


Swing Dev>[9] Review Request JDK-8161913 [PIT] java/awt/Window/8159168/SetShapeTest.java mostly fails

2016-08-16 Thread Rajeev Chamyal
Hello All,

 

Please review the following webrev.

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

Webrev : http://cr.openjdk.java.net/~rchamyal/8161913/webrev.00/

 

Issue : Test was failing when run repeatedly.

 

Fix: Added delay in test.

 

Regards,

Rajeev Chamyal