Hi Sergey, The fix looks good to me.
Thanks, Dmitry > On 7 Mar 2019, at 06:59, Sergey Bylokhov <[email protected]> wrote: > > Thank you for review! > Looking for the second +1 from someone else. > > On 20/02/2019 22:20, Krishna Addepalli wrote: >> +1 >> Krishna >> -----Original Message----- >> From: Sergey Bylokhov >> Sent: Thursday, February 21, 2019 8:01 AM >> To: [email protected] >> Subject: <AWT Dev> [13] Review Request: 8219504 Test for JDK-8211435 can be >> run on all platforms >> Hello. >> Please review the fix for jdk 13. >> Bug: https://bugs.openjdk.java.net/browse/JDK-8219504 >> Fix: http://cr.openjdk.java.net/~serb/8219504/webrev.00 >> Currently the test for JDK-8211435 is executed on macOS only, because of >> JDK-8204142[1][2]: >> But since JDK-8204142 was fixed we can run it on all platforms. >> [1] https://bugs.openjdk.java.net/browse/JDK-8204142 >> [2] http://mail.openjdk.java.net/pipermail/awt-dev/2018-October/014492.html > > > -- > Best regards, Sergey.
