<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    Little remainder.<br>
    <br>
    <div class="moz-cite-prefix">On 06.05.2016 21:14, Svetlana
      Nikandrova wrote:<br>
    </div>
    <blockquote cite="mid:572CDEEE.80108@oracle.com" type="cite">
      <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
      Hello,<br>
      <br>
      I would be grateful if someone could find time to review this
      backport of tests enhancement.<br>
      I received few off-line notes and decided to rework this backport
      a little.<br>
      <br>
      Here is my new webrev:<br>
      <a moz-do-not-send="true"
        href="http://cr.openjdk.java.net/%7Esnikandrova/8049171/webrev.01/">http://cr.openjdk.java.net/~snikandrova/8049171/webrev.01/</a><br>
      <br>
      List of changes from original patch:<br>
      1) No ProcessTools.java changes in webrev as they have been
      backported earlier. <br>
      2) Utils.java : new utility method  <span class="new">transferBetweenStreams</span>
      meant to be used instead of InputStream's transferTo method that
      is present since 9. <br>
      3)
      <meta http-equiv="content-type" content="text/html; charset=UTF-8">
      JarUtils.java: changes in createJar method to consider a situation
      then files aren't located in current directory.<br>
      4) NestedActions.java: refactored to use JarUtils method to create
      jars.<br>
      5) TimestampCheck.java - omitted
      System.getProperty("test.tool.vm.opts") that is not used in jdk 8
      testing. <br>
      6) Used System.getProperty("test.jdk") instead of
      System.getProperty("java.home") in tests.<br>
      <br>
      Thank you, <br>
      Svetlana <br>
        <br>
      <div class="moz-cite-prefix">On 28.04.2016 20:35, Svetlana
        Nikandrova wrote:<br>
      </div>
      <blockquote cite="mid:572249EB.8090808@oracle.com" type="cite">Hello,

        <br>
        <br>
        please review and approve the backport of tests enhancement: <br>
        Original tests from <br>
        <a moz-do-not-send="true" class="moz-txt-link-freetext"
          href="https://bugs.openjdk.java.net/browse/JDK-8049171">https://bugs.openjdk.java.net/browse/JDK-8049171</a>
        <br>
        jdk 9 review:
        <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-January/031006.html">http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-January/031006.html</a><br>
        jdk 9 changeset: <a moz-do-not-send="true"
          class="moz-txt-link-freetext"
          href="http://hg.openjdk.java.net/jdk9/dev/jdk/rev/5c61ccd9c162">http://hg.openjdk.java.net/jdk9/dev/jdk/rev/5c61ccd9c162</a>
        <br>
        Originally reviewed by Max Wang <br>
        <br>
        Webrev: <br>
        <a moz-do-not-send="true" class="moz-txt-link-freetext"
          href="http://cr.openjdk.java.net/%7Esnikandrova/8049171/webrev.00/">http://cr.openjdk.java.net/~snikandrova/8049171/webrev.00/</a>
        <a moz-do-not-send="true" class="moz-txt-link-rfc2396E"
          href="http://cr.openjdk.java.net/%7Esnikandrova/8049171/webrev.00/"><http://cr.openjdk.java.net/%7Esnikandrova/8049171/webrev.00/></a>
        <br>
        <br>
        List of changes between original patch and current webrev: <br>
        <br>
        1) No ProcessTools.java changes as they have been backported
        earlier. <br>
        2) JarUtils.java : private method transferToOutputStream instead
        ofInputStream's transferTo method (present since 9). <br>
        3) TimestampCheck.java - changes applied by hand to omit
        System.getProperty("test.tool.vm.opts") that is not used in jdk
        8 testing <br>
        4) Used System.getProperty("test.jdk") instead of
        System.getProperty("java.home") <br>
        <br>
        Tested with JPRT. <br>
        <br>
        Thank you, <br>
        Svetlana <br>
      </blockquote>
      <br>
    </blockquote>
    <br>
  </body>
</html>