Re: [9] Review Request: 8164104 Cleanup of javaclient related mapfiles

2016-08-17 Thread Sergey Bylokhov
Tested on solarisX64/solarisSparc/linuxX64 no issues were found(SwingSet2/Java2Demo). But I wonder why the text on solarissparc filled by yellow shadow(Is it a known issue)? On 16.08.16 19:33, Phil Race wrote: I think "JPRT is OK" is good but may be not enough here? I am not sure if the build

Re: Review Request: JDK-8144735 [hidpi] javax/swing/JWindow/ShapedAndTranslucentWindows/TranslucentPerPixelTranslucentGradient.java fails

2016-08-17 Thread Alexandr Scherbatiy
On 8/17/2016 11:30 AM, Prem Balakrishnan wrote: Hi Alexandr, Thankyou for the review. YES scaled SurfaceData returns proper scale values from getDefaultScaleX()/getDefaultScaleY(), which I have used in the current patch. Please review the updated patch http://cr.openjdk.java.net/~pkbalakr

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

2016-08-17 Thread Avik Niyogi
Looks good to me. With Regards, Avik Niyogi > From: Alexandr Scherbatiy > Sent: 16 August 2016 22:55 > To: Rajeev Chamyal; Sergey Bylokhov; awt-dev@openjdk.java.net > > Subject: Re: Swing Dev>[9] Review Request JDK-8161913 [PIT] > java/awt/Window/8159168/SetSh

Re: Review Request: JDK-8144735 [hidpi] javax/swing/JWindow/ShapedAndTranslucentWindows/TranslucentPerPixelTranslucentGradient.java fails

2016-08-17 Thread Prem Balakrishnan
Hi Alexandr, Thankyou for the review. YES scaled SurfaceData returns proper scale values from getDefaultScaleX()/getDefaultScaleY(), which I have used in the current patch. Please review the updated patch http://cr.openjdk.java.net/~pkbalakr/8144735/webrev.02/ Regards, Prem From: Alexa

Re: [9] Review request for 8162840: Desktop. enableSuddenTermination() has no effect

2016-08-17 Thread Yuri Nesterenko
In fact, we have a bunch of them in the func. workspace, all more or less easily portable to jtreg. I'll move them in a bulk, time permitting. -yan On 08/16/2016 07:47 PM, Alexander Zvegintsev wrote: Sure, but it is already created by Yuri and attached to the issue. I can include it with this