[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-swing-dev
Subject:    Re: <Swing Dev> 8163161: [PIT][TEST_BUG] increase timeout in javax/swing/plaf/nimbus/8057791/bug8057
From:       Avik Niyogi <avik.niyogi () oracle ! com>
Date:       2016-08-24 8:55:42
Message-ID: 574B95B8-4B53-43CC-B10D-ABEAFAA9F2AB () oracle ! com
[Download RAW message or body]

Hi All,
A gentle reminder, please review my code change.

With Regards,
Avik Niyogi
> On 23-Aug-2016, at 2:28 pm, Alexander Scherbatiy <alexandr.scherbatiy@oracle.com> \
> wrote: 
> The fix looks good to me.
> 
> Thanks,
> Alexandr.
> 
> On 23/08/16 10:38, Avik Niyogi wrote:
> > Hi All,
> > 
> > Kindly review the fix for JDK9. 
> > 
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8163161 \
> > <https://bugs.openjdk.java.net/browse/JDK-8163161> 
> > Webrev: http://cr.openjdk.java.net/~aniyogi/8163161/webrev.00/ \
> > <http://cr.openjdk.java.net/%7Eaniyogi/8163161/webrev.00/> 
> > Issue: This test javax/swing/plaf/nimbus/8057791/bug8057791.java times out
> > 
> > Cause:  No timeout delay was provided so it would fail on other systems.
> > 
> > Fix: Appropriate changes for timeout were added.
> > 
> > With Regards,
> > Avik Niyogi
> 


[Attachment #3 (unknown)]

<html><head><meta http-equiv="Content-Type" content="text/html \
charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: \
space; -webkit-line-break: after-white-space;" class="">Hi All,<div class="">A gentle \
reminder, please review my code change.</div><div class=""><br class=""></div><div \
class="">With Regards,</div><div class="">Avik Niyogi<br class=""><div><blockquote \
type="cite" class=""><div class="">On 23-Aug-2016, at 2:28 pm, Alexander Scherbatiy \
&lt;<a href="mailto:alexandr.scherbatiy@oracle.com" \
class="">alexandr.scherbatiy@oracle.com</a>&gt; wrote:</div><br \
class="Apple-interchange-newline"><div class="">  
    <meta content="text/html; charset=windows-1252" http-equiv="Content-Type" \
class="">  
  <div bgcolor="#FFFFFF" text="#000000" class="">
    The fix looks good to me.<br class="">
    <br class="">
    Thanks,<br class="">
    Alexandr.<br class="">
    <br class="">
    <div class="moz-cite-prefix">On 23/08/16 10:38, Avik Niyogi wrote:<br class="">
    </div>
    <blockquote cite="mid:52DF3D01-F9D5-466C-B845-8AF70508D764@oracle.com" \
type="cite" class="">  <meta http-equiv="Content-Type" content="text/html;
        charset=windows-1252" class="">
      <div class="">Hi All,</div>
      <div class="">
        <div class="">
          <div class=""><br class="">
          </div>
          Kindly review the fix for JDK9.&nbsp;
          <div class=""><br class="">
          </div>
          <div class=""><b class="">Bug:&nbsp;<a moz-do-not-send="true" \
href="https://bugs.openjdk.java.net/browse/JDK-8163161" \
class="">https://bugs.openjdk.java.net/browse/JDK-8163161</a></b></div>  <div \
class=""><br class="">  </div>
          <div class=""><b class=""><span class="">Webrev:&nbsp;<a \
moz-do-not-send="true" \
href="http://cr.openjdk.java.net/%7Eaniyogi/8163161/webrev.00/" \
class="">http://cr.openjdk.java.net/~aniyogi/8163161/webrev.00/</a></span></b></div>  \
<div class=""><br class="">  </div>
          <div class=""><b class="">Issue:&nbsp;</b>This test
            javax/swing/plaf/nimbus/8057791/bug8057791.java times out</div>
          <div class=""><br class="">
          </div>
          <div class=""><b class="">Cause:&nbsp;</b>&nbsp;No timeout delay was
            provided so it would fail on other systems.</div>
          <div class=""><br class="">
          </div>
          <div class=""><b class="">Fix:</b><span class="">&nbsp;Appropriate
              changes for timeout were added.</span></div>
          <div class=""><br class="">
          </div>
          <div class="">With Regards,</div>
          <div class="">Avik Niyogi</div>
        </div>
      </div>
    </blockquote>
    <br class="">
  </div>

</div></blockquote></div><br class=""></div></body></html>



[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic