<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <meta http-equiv="content-type" content="text/html;
      charset=ISO-8859-1">
    I don't understand why the relative path is used  in some two cases
    and<br>
    an absolute path in others. OK you didn't change that but does it
    make<br>
    sense to standardise on what I hope is the absolute path which is
    easier<br>
    to get right and will withstand the tests being moved ? <br>
    Can we do the same for "helpers" ?<br>
    <br>
    -phil.<br>
    <br>
test/jdk/java/awt/Modal/ModalExclusionTests/ApplicationExcludeDialogFileTest.java<br>
    <br>
    @@ -29,11 +29,11 @@<br>
      * @bug 8047179 8044429<br>
      * @summary Check whether a FileDialog blocks an application
    modality excluded Dialog<br>
      *          (it shouldn't). Checks also whether setting a parent
    dialog to be<br>
      *          modality excluded excludes its children from being
    blocked too.<br>
      *<br>
    - * @library ../helpers /lib/testlibrary/<br>
    + * @library ../helpers /lib/client/<br>
      * @library /test/lib<br>
      * @build ExtendedRobot<br>
      * @build Flag<br>
      * @build TestDialog<br>
      * @build TestWindow<br>
    <br>
    <meta http-equiv="content-type" content="text/html;
      charset=ISO-8859-1">
test/jdk/java/awt/Modal/ModalDialogOrderingTest/ModalDialogOrderingTest.java<br>
    <br>
    @@ -33,11 +33,11 @@<br>
      * @test<br>
      * @key headful<br>
      * @bug 8008728<br>
      * @summary [macosx] Swing. JDialog. Modal dialog goes to
    background<br>
      * @author Alexandr Scherbatiy<br>
    - * @library ../../../../lib/testlibrary<br>
    + * @library ../../../../lib/client<br>
      * @build ExtendedRobot<br>
      * @run main ModalDialogOrderingTest<br>
      */<br>
     public class ModalDialogOrderingTest {<br>
     <br>
    <br>
    -phil.<br>
    <br>
    On 11/6/18, 5:21 PM, Alexandre (Shura) Iline wrote:
    <blockquote
      cite="mid:78863B1F-9B4C-4507-BBFC-680965D62984@oracle.com"
      type="cite">
      <pre wrap="">Ping.

</pre>
      <blockquote type="cite">
        <pre wrap="">On Oct 29, 2018, at 3:02 PM, Alexandre (Shura) Iline <a class="moz-txt-link-rfc2396E" href="mailto:alexandre.iline@oracle.com"><alexandre.iline@oracle.com></a> wrote:

Hi.

Please take a look on this fix for JDK-8186549.

For testing I have selected tests and groups of tests which use the ExtendedRobot class. The tests show the same pass rate. There are still few tests which use lib/testlibrary, they are not affected by this fix.

Task: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8186549">https://bugs.openjdk.java.net/browse/JDK-8186549</a>
Webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~shurailine/8186549/webrev.00">http://cr.openjdk.java.net/~shurailine/8186549/webrev.00</a>
Tests used for testing the fix: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~shurailine/8186549/extended_robot_tests">http://cr.openjdk.java.net/~shurailine/8186549/extended_robot_tests</a>

Shura

</pre>
      </blockquote>
      <pre wrap="">
</pre>
    </blockquote>
  </body>
</html>