<Swing Dev> Review request for JDK-8131751: Test javax/swing/plaf/gtk/crash/RenderBadPictureCrash.java fails UnsupportedOperationException

[Alexandr Scherbatiy]

  The fix looks good to me.

   Thanks,
   Alexandr.

On 2/1/2016 4:32 AM, Vikrant Agarwal wrote:
>
> Thanks Alexander, I have updated the webrev for the same.
>
> Updated Webrev: 
> http://cr.openjdk.java.net/~ntv/vikrant/8131751/webrev.01/ 
> <http://cr.openjdk.java.net/%7Entv/vikrant/8131751/webrev.01/>
>
> *Bug:* https://bugs.openjdk.java.net/browse/JDK-8131751
>
> Best Regards,
>
> Vikrant
>
> *From:*Alexander Scherbatiy
> *Sent:* Thursday, January 28, 2016 11:13 PM
> *To:* Vikrant Agarwal; swing-dev at openjdk.java.net
> *Subject:* Re: <Swing Dev> Review request for JDK-8131751: Test 
> javax/swing/plaf/gtk/crash/RenderBadPictureCrash.java fails 
> UnsupportedOperationException
>
>
> The fix looks good to me.
>
> Just a small note: it could be better to get the graphics device from 
> the frame itself (f.getGraphicsConfiguration().getDevice()).
>
> Thanks,
> Alexandr.
>
> On 1/28/2016 2:32 AM, Vikrant Agarwal wrote:
>
>     Hi All,
>
>     Kindly review the fix for JDK9.
>
>     *Bug:* https://bugs.openjdk.java.net/browse/JDK-8131751
>
>     *Webrev:* http://cr.openjdk.java.net/~srastogi/8131751/webrev.00/
>     <http://cr.openjdk.java.net/%7Esrastogi/8131751/webrev.00/>
>
>     *Issue:* [TEST_BUG] Test
>     javax/swing/plaf/gtk/crash/RenderBadPictureCrash.java fails
>     UnsupportedOperationException
>
>     *Cause:* test failed on those systems where PERPIXEL_TRANSLUCENT
>     translucency is not supported as it is doing  f.setBackground(new
>     Color(0, 0, 0, 0)); (making alpha value 0)
>
>     *Fix:* Added a check for checking support for PERPIXEL_TRANSLUCENT
>     translucency before f.setBackground(new Color(0, 0, 0, 0));
>
>     Best Regards,
>
>     Vikrant Agarwal
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20160201/05f37b5c/attachment.html>