Review request for JDK-8133027: [Jigsaw] Test java/awt/PrintJob/Text/stringwidth.sh fails during compilation

2015-09-03 Thread Renjith Alexander
Hi all, Please review the fix for the bug: JDK-8133027: [Jigsaw] Test java/awt/PrintJob/Text/stringwidth.sh fails during compilation BUG ID - https://bugs.openjdk.java.net/browse/JDK-8133027 The webrev is: http://cr.openjdk.java.net/~pchopra/8133027/webrev.02/ One test was moved from closed

Re: [9, 8] Review request for 8081485: EDT auto shutdown is broken in case of new event queue usage

2015-09-03 Thread Anton Litvinov
Hello Sergey and Artem, After Anton Nashatyrev left the organization, I was assigned to this bug to complete work on it. Though Sergey approved the 1st version of the fix, the second approval was not received, and the fix was not integrated into "jdk9/client". Today I have compiled "jdk9/clien

Re: [9] Review Request for 8130471: [Regression] Test java/awt/Mouse/MouseModifiersUnitTest/MouseModifiersUnitTest_Extra.java fails

2015-09-03 Thread Sergey Bylokhov
Looks fine. On 03.09.15 17:50, Semyon Sadetsky wrote: Any other reviewers? Alexander, Sergey? On 8/26/2015 4:34 PM, Alexander Scherbatiy wrote: The fix looks good to me. Thanks, Alexandr. On 8/4/2015 1:22 PM, Semyon Sadetsky wrote: On 7/29/2015 2:06 PM, Alexander Scherbatiy wrote: On 7

Re: [9] Review Request for 8131670: [TEST_BUG]Test java/awt/EventQueue/6980209/bug6980209.java fails on Linux

2015-09-03 Thread Sergey Bylokhov
Looks fine. On 03.09.15 17:57, Semyon Sadetsky wrote: Guys, just a reminder. On 8/4/2015 3:34 PM, Semyon Sadetsky wrote: Hi Sergey, It was not me. webrev diff to the last repository version. the updated fix: http://cr.openjdk.java.net/~ssadetsky/8131670/webrev.01/ --Semyon http://cr.openjdk

Re: [9] Review Request for 8131670: [TEST_BUG]Test java/awt/EventQueue/6980209/bug6980209.java fails on Linux

2015-09-03 Thread Semyon Sadetsky
Guys, just a reminder. On 8/4/2015 3:34 PM, Semyon Sadetsky wrote: Hi Sergey, It was not me. webrev diff to the last repository version. the updated fix: http://cr.openjdk.java.net/~ssadetsky/8131670/webrev.01/ --Semyon http://cr.openjdk.java.net/~ssadetsky/8131670/webrev.01/ On 7/22/2015 12

Re: [9] Review Request for 8130471: [Regression] Test java/awt/Mouse/MouseModifiersUnitTest/MouseModifiersUnitTest_Extra.java fails

2015-09-03 Thread Semyon Sadetsky
Any other reviewers? Alexander, Sergey? On 8/26/2015 4:34 PM, Alexander Scherbatiy wrote: The fix looks good to me. Thanks, Alexandr. On 8/4/2015 1:22 PM, Semyon Sadetsky wrote: On 7/29/2015 2:06 PM, Alexander Scherbatiy wrote: On 7/23/2015 9:21 PM, Semyon Sadetsky wrote: Hello, Please

Re: [9] Review Request for 8132664: closed/javax/swing/DataTransfer/DefaultNoDrop/DefaultNoDrop.java locks on Windows

2015-09-03 Thread Semyon Sadetsky
On 8/5/2015 2:33 PM, Sergey Bylokhov wrote: On 05.08.15 14:20, Semyon Sadetsky wrote: On 8/5/2015 1:39 PM, Sergey Bylokhov wrote: On 05.08.15 13:18, Semyon Sadetsky wrote: On 8/5/2015 12:27 PM, Sergey Bylokhov wrote: On 04.08.15 14:54, Semyon Sadetsky wrote: On 8/3/2015 6:05 PM, Sergey

Re: [awt-dev] Request for Approval and Review: JDK-8133027: [Jigsaw] Test java/awt/PrintJob/Text/stringwidth.sh fails during compilation

2015-09-03 Thread Alexander Scherbatiy
Hi Renjith, It seems that you just replied to an another email and the request was just filtered out. See http://mail.openjdk.java.net/pipermail/awt-dev/2015-August/thread.html You need to create a new email and resend the review request. Thanks, Alexandr. On 8/31/2015 8:03 AM, Renjit

Re: [9] Review Request for 8014725: closed/java/awt/Clipboard/HTMLTransferTest/HTMLTransferTest.html failed intermittently

2015-09-03 Thread Semyon Sadetsky
Alexanders? Any questions? On 8/5/2015 3:11 PM, Semyon Sadetsky wrote: Hello, Please review fix for JDK9: bug: https://bugs.openjdk.java.net/browse/JDK-8014725 webrev: http://cr.openjdk.java.net/~ssadetsky/8014725/webrev.00/ The test is moved from closed to open repo. Initially it was a test

[awt-dev] Request for Review: JDK-8133539 - [TEST_BUG] Split java/awt/image/MultiResolutionImageTest.java in two to allow restricted access

2015-09-03 Thread Renjith Alexander
Hi all, Please review the fix for the bug:  JDK-8133539 - [TEST_BUG] Split java/awt/image/MultiResolutionImageTest.java in two to allow restricted access BUG ID - https://bugs.openjdk.java.net/browse/JDK-8133539 The webrev is: http://cr.openjdk.java.net/~pchopra/8133539/webrev.01/webrev/     --