<Swing Dev> [9] Review request for 8081478: [TESTBUG] Test javax/swing/plaf/synth/SynthButtonUI/6276188/bug6276188.java fails for OEL 7 only

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

<Swing Dev> [9] Review request for 8081478: [TESTBUG] Test javax/swing/plaf/synth/SynthButtonUI/6276188/bug6276188.java fails for OEL 7 only

Semyon Sadetsky
Hello,

Please review fix for JDK9:

bug: https://bugs.openjdk.java.net/browse/JDK-8081478

webrev: http://cr.openjdk.java.net/~ssadetsky/8081478/webrev.00/

Delay was added to ensure that window is shown to take it final location.

--Semyon

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: <Swing Dev> [9] Review request for 8081478: [TESTBUG] Test javax/swing/plaf/synth/SynthButtonUI/6276188/bug6276188.java fails for OEL 7 only

Sergey Bylokhov
Looks fine.

>
> Hello,
>
> Please review fix for JDK9:
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8081478
>
> webrev: http://cr.openjdk.java.net/~ssadetsky/8081478/webrev.00/
>
> Delay was added to ensure that window is shown to take it final location.
>
> --Semyon
>

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: <Swing Dev> [9] Review request for 8081478: [TESTBUG] Test javax/swing/plaf/synth/SynthButtonUI/6276188/bug6276188.java fails for OEL 7 only

Yuri Nesterenko
In reply to this post by Semyon Sadetsky
+1

-yan

On 04/19/2017 08:27 PM, Semyon Sadetsky wrote:

> Hello,
>
> Please review fix for JDK9:
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8081478
>
> webrev: http://cr.openjdk.java.net/~ssadetsky/8081478/webrev.00/
>
> Delay was added to ensure that window is shown to take it final location.
>
> --Semyon
>

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: <Swing Dev> [9] Review request for 8081478: [TESTBUG] Test javax/swing/plaf/synth/SynthButtonUI/6276188/bug6276188.java fails for OEL 7 only

Alexandr Scherbatiy
In reply to this post by Semyon Sadetsky

   The fix looks good to me.

   Thanks,
   Alexandr.

On 4/19/2017 8:27 PM, Semyon Sadetsky wrote:

> Hello,
>
> Please review fix for JDK9:
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8081478
>
> webrev: http://cr.openjdk.java.net/~ssadetsky/8081478/webrev.00/
>
> Delay was added to ensure that window is shown to take it final location.
>
> --Semyon
>

Loading...