Hello Yuri, Can you please review below webrev. Webrev: http://cr.openjdk.java.net/~rchamyal/8163160/webrev.00/
Regards, Rajeev Chamyal -----Original Message----- From: Rajeev Chamyal Sent: 16 August 2016 18:45 To: Sergey Bylokhov; Alexander Scherbatiy; swing-dev@openjdk.java.net Subject: Re: <Swing Dev> Swing Dev>[9] Review Request JDK-8163160 [PIT][TEST_BUG] Some issues in java/awt/image/multiresolution/MultiResolutionIcon/IconTest.java 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.