Re: [9] Review request for JDK-8153282 [TEST_BUG] some new JInternalFrame tests fail

2016-05-04 Thread Alexander Scherbatiy


  The fix looks good to me.

  Thanks,
  Alexandr.

On 5/2/2016 6:20 PM, Rajeev Chamyal wrote:

Hello Sergey,

Checked the tests with 1.9.0-ea-b86 build.
All tests failed.

Regards,
Rajeev Chamyal

-Original Message-
From: Sergey Bylokhov
Sent: 02 May 2016 19:05
To: Rajeev Chamyal; swing-dev@openjdk.java.net; Avik Niyogi
Subject: Re:  [9] Review request for JDK-8153282 [TEST_BUG] some new 
JInternalFrame tests fail

Looks fine, please confirm that the tests still fails before corresponding bugs 
were fixed.

On 02.05.16 13:19, Rajeev Chamyal wrote:

Hello All,



Please review the following fix.



Bug: https://bugs.openjdk.java.net/browse/JDK-8153282

Webrev: http://cr.openjdk.java.net/~rchamyal/8153282/webrev.00/



Fix: Added delay for test TestJInternalFrameMaximize.java and
TestJInternalFrameMinimize.java

Increased the time value for TestJInternalFrameDispose.java



Test are running fine on Mac,Windows and linux after the fix.



Regards,

Rajeev Chamyal





--
Best regards, Sergey.




Re: [9] Review request for JDK-8153282 [TEST_BUG] some new JInternalFrame tests fail

2016-05-04 Thread Avik Niyogi
Looks fine.

> On 03-May-2016, at 12:48 am, Sergey Bylokhov  
> wrote:
> 
> Thanks.
> On 02.05.16 18:20, Rajeev Chamyal wrote:
>> Hello Sergey,
>> 
>> Checked the tests with 1.9.0-ea-b86 build.
>> All tests failed.
>> 
>> Regards,
>> Rajeev Chamyal
>> 
>> -Original Message-
>> From: Sergey Bylokhov
>> Sent: 02 May 2016 19:05
>> To: Rajeev Chamyal; swing-dev@openjdk.java.net; Avik Niyogi
>> Subject: Re:  [9] Review request for JDK-8153282 [TEST_BUG] some 
>> new JInternalFrame tests fail
>> 
>> Looks fine, please confirm that the tests still fails before corresponding 
>> bugs were fixed.
>> 
>> On 02.05.16 13:19, Rajeev Chamyal wrote:
>>> Hello All,
>>> 
>>> 
>>> 
>>> Please review the following fix.
>>> 
>>> 
>>> 
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8153282
>>> 
>>> Webrev: http://cr.openjdk.java.net/~rchamyal/8153282/webrev.00/
>>> 
>>> 
>>> 
>>> Fix: Added delay for test TestJInternalFrameMaximize.java and
>>> TestJInternalFrameMinimize.java
>>> 
>>> Increased the time value for TestJInternalFrameDispose.java
>>> 
>>> 
>>> 
>>> Test are running fine on Mac,Windows and linux after the fix.
>>> 
>>> 
>>> 
>>> Regards,
>>> 
>>> Rajeev Chamyal
>>> 
>>> 
>>> 
>> 
>> 
>> --
>> Best regards, Sergey.
>> 
> 
> 
> -- 
> Best regards, Sergey.



Re: [9] Review request for JDK-8153282 [TEST_BUG] some new JInternalFrame tests fail

2016-05-02 Thread Sergey Bylokhov

Thanks.
On 02.05.16 18:20, Rajeev Chamyal wrote:

Hello Sergey,

Checked the tests with 1.9.0-ea-b86 build.
All tests failed.

Regards,
Rajeev Chamyal

-Original Message-
From: Sergey Bylokhov
Sent: 02 May 2016 19:05
To: Rajeev Chamyal; swing-dev@openjdk.java.net; Avik Niyogi
Subject: Re:  [9] Review request for JDK-8153282 [TEST_BUG] some new 
JInternalFrame tests fail

Looks fine, please confirm that the tests still fails before corresponding bugs 
were fixed.

On 02.05.16 13:19, Rajeev Chamyal wrote:

Hello All,



Please review the following fix.



Bug: https://bugs.openjdk.java.net/browse/JDK-8153282

Webrev: http://cr.openjdk.java.net/~rchamyal/8153282/webrev.00/



Fix: Added delay for test TestJInternalFrameMaximize.java and
TestJInternalFrameMinimize.java

Increased the time value for TestJInternalFrameDispose.java



Test are running fine on Mac,Windows and linux after the fix.



Regards,

Rajeev Chamyal






--
Best regards, Sergey.




--
Best regards, Sergey.


Re: [9] Review request for JDK-8153282 [TEST_BUG] some new JInternalFrame tests fail

2016-05-02 Thread Rajeev Chamyal
Hello Sergey,

Checked the tests with 1.9.0-ea-b86 build.
All tests failed.

Regards,
Rajeev Chamyal

-Original Message-
From: Sergey Bylokhov 
Sent: 02 May 2016 19:05
To: Rajeev Chamyal; swing-dev@openjdk.java.net; Avik Niyogi
Subject: Re:  [9] Review request for JDK-8153282 [TEST_BUG] some new 
JInternalFrame tests fail

Looks fine, please confirm that the tests still fails before corresponding bugs 
were fixed.

On 02.05.16 13:19, Rajeev Chamyal wrote:
> Hello All,
>
>
>
> Please review the following fix.
>
>
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8153282
>
> Webrev: http://cr.openjdk.java.net/~rchamyal/8153282/webrev.00/
>
>
>
> Fix: Added delay for test TestJInternalFrameMaximize.java and 
> TestJInternalFrameMinimize.java
>
> Increased the time value for TestJInternalFrameDispose.java
>
>
>
> Test are running fine on Mac,Windows and linux after the fix.
>
>
>
> Regards,
>
> Rajeev Chamyal
>
>
>


--
Best regards, Sergey.


Re: [9] Review request for JDK-8153282 [TEST_BUG] some new JInternalFrame tests fail

2016-05-02 Thread Sergey Bylokhov
Looks fine, please confirm that the tests still fails before 
corresponding bugs were fixed.


On 02.05.16 13:19, Rajeev Chamyal wrote:

Hello All,



Please review the following fix.



Bug: https://bugs.openjdk.java.net/browse/JDK-8153282

Webrev: http://cr.openjdk.java.net/~rchamyal/8153282/webrev.00/



Fix: Added delay for test TestJInternalFrameMaximize.java and
TestJInternalFrameMinimize.java

Increased the time value for TestJInternalFrameDispose.java



Test are running fine on Mac,Windows and linux after the fix.



Regards,

Rajeev Chamyal






--
Best regards, Sergey.


[9] Review request for JDK-8153282 [TEST_BUG] some new JInternalFrame tests fail

2016-05-02 Thread Rajeev Chamyal
Hello All,

 

Please review the following fix.

 

Bug: https://bugs.openjdk.java.net/browse/JDK-8153282

Webrev: http://cr.openjdk.java.net/~rchamyal/8153282/webrev.00/

 

Fix: Added delay for test TestJInternalFrameMaximize.java and 
TestJInternalFrameMinimize.java

Increased the time value for TestJInternalFrameDispose.java

 

Test are running fine on Mac,Windows and linux after the fix.

 

Regards,

Rajeev Chamyal