[9] Review request for 8166897: Some font overlap in the Optionpane dialog.

2016-10-12 Thread Semyon Sadetsky
Hello, Please review fix for JDK9: bug: https://bugs.openjdk.java.net/browse/JDK-8166897 webrev: http://cr.openjdk.java.net/~ssadetsky/8166897/webrev.00/ The setResizable() method of XDecoratedPeer class clears the frame insets to allow them to receive the updated size. In some situations

Re: JDK-8098530: ArrayIndexOutOfBoundsException at sun.awt.X11.XEvent.getFieldsAsString

2016-10-12 Thread Sergey Bylokhov
Is it possible to create a test for this issue? On 12.10.16 19:07, Sergey Malenkov wrote: Hi all, Could you please consider the following changes to fix JDK-8098530? --- src/solaris/classes/sun/awt/X11/generator/WrapperGenerator.java (revision f0ae41314e223bf1b97c8126ac2b0d6edb5ee99d) +++

JDK-8098530: ArrayIndexOutOfBoundsException at sun.awt.X11.XEvent.getFieldsAsString

2016-10-12 Thread Sergey Malenkov
Hi all, Could you please consider the following changes to fix JDK-8098530? --- src/solaris/classes/sun/awt/X11/generator/WrapperGenerator.java (revision f0ae41314e223bf1b97c8126ac2b0d6edb5ee99d) +++ src/solaris/classes/sun/awt/X11/generator/WrapperGenerator.java (revision ) @@ -690,7 +690,7 @@

Re: [9] Review request for JDK-8161473:[TEST_BUG] [macosx] add autodelay to java/awt/TrayIcon/TrayIconEventModifiers/TrayIconEventModifiersTest.java

2016-10-12 Thread Sergey Bylokhov
On 12.10.16 9:04, Ambarish Rapte wrote: Thanks Anubhav, Changes look fine to me. +1 *From:*Anubhav Meena *Sent:* Monday, October 10, 2016 5:06 PM *To:* Ambarish Rapte *Cc:* awt-dev@openjdk.java.net *Subject:* Re: [9] Review request for JDK-8161473:[TEST_BUG] [macosx] add

Re: [9] Review request for JDK-8158380: Regression: java/awt/List/ActionEventTest/ActionEventTest.java

2016-10-12 Thread Sergey Bylokhov
Looks fine. On 12.10.16 14:40, Manajit Halder wrote: Thanks you for your comment. Copyright is corrected in both the tests. Please review the modified webrev. http://cr.openjdk.java.net/~mhalder/8158380/webrev.02/ Thanks, Manajit On 10-Oct-2016, at 5:58 pm, Sergey Bylokhov

Re: [9] Fix for JDK-8166980 : JCK testing of Window.setIconImage() leads to VM crash starting approx from JDK9 b134

2016-10-12 Thread Sergey Bylokhov
Looks fine. On 12.10.16 10:10, Ajit Ghaisas wrote: Hi, Bug : https://bugs.openjdk.java.net/browse/JDK-8166980 Issue and Fix : Fix for JDK-8147648 needs to be corrected a little. IconInfo::setScaledSize(int width, int

Re: [9] Review request for JDK-8161473:[TEST_BUG] [macosx] add autodelay to java/awt/TrayIcon/TrayIconEventModifiers/TrayIconEventModifiersTest.java

2016-10-12 Thread Anubhav Meena
Hi Sergey, Please review the webrev and provide your valuable feedback. http://cr.openjdk.java.net/~rchamyal/anmeena/8161473/webrev.01/ Thanks, Anubhav > On Oct 12, 2016, at 11:34 AM, Ambarish Rapte

Re: [9] Review request for JDK-8158380: Regression: java/awt/List/ActionEventTest/ActionEventTest.java

2016-10-12 Thread Manajit Halder
Thanks you for your comment. Copyright is corrected in both the tests. Please review the modified webrev. http://cr.openjdk.java.net/~mhalder/8158380/webrev.02/ Thanks, Manajit > On 10-Oct-2016, at 5:58 pm, Sergey Bylokhov

Re: RFR: 8167523: JDK 9 build failure on MacOS due to unhandled cases in switch statement

2016-10-12 Thread Sergey Malenkov
> Looks fine. +1 Thanks! Sergey A. Malenkov

Re: [9] Review request for 8167486: Device.getDisplayMode() doesn't report refresh rate on Linux in case of dual screen

2016-10-12 Thread Semyon Sadetsky
On 10/11/2016 5:02 PM, Sergey Bylokhov wrote: It seems that this bug is a duplicate of JDK-8159953? yes, I have closed it. On 11.10.16 16:40, Sergey Bylokhov wrote: Should toString() method report zero HZ as unknown(os something like that) instead of 0? And probably "DEPTH_MULTI"also should

[9] Fix for JDK-8166980 : JCK testing of Window.setIconImage() leads to VM crash starting approx from JDK9 b134

2016-10-12 Thread Ajit Ghaisas
Hi, Bug : https://bugs.openjdk.java.net/browse/JDK-8166980 Issue and Fix : Fix for HYPERLINK "https://bugs.openjdk.java.net/browse/JDK-8147648"JDK-8147648 needs to be corrected a little. IconInfo::setScaledSize(int width, int height) method sets scaledWidth,