<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Shouldn't this use reference equality ?<br>
    Also the test does a number of wild card imports instead of<br>
    the specific classes it needs. And I am not sure why it<br>
    is importing java.awt.* anyway ..<br>
    <br>
    And I find qualifying "java.lang.String" to be unnecessary.<br>
    <br>
    -phil.<br>
    <br>
    On 11/3/15, 12:59 AM, prasanta sadhukhan wrote:
    <blockquote cite="mid:56387767.7060101@oracle.com" type="cite">
      <meta http-equiv="content-type" content="text/html; charset=UTF-8">
      Hi All,<br>
      <br>
      Please review a fix for jdk9.<br>
      <br>
      Bug: <a moz-do-not-send="true" class="moz-txt-link-freetext"
        href="https://bugs.openjdk.java.net/browse/JDK-8067059">https://bugs.openjdk.java.net/browse/JDK-8067059</a><br>
      webrev: <a moz-do-not-send="true" class="moz-txt-link-freetext"
        href="http://cr.openjdk.java.net/%7Epsadhukhan/8067059/webrev.00/">http://cr.openjdk.java.net/~psadhukhan/8067059/webrev.00/</a><br>
      <br>
      The issue was if we "cancel" Printer Job setup dialog which has
      been created with DialogSelectionType.NATIVE <br>
      it should return "null" as per spec <br>
      <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://docs.oracle.com/javase/8/docs/api/java/awt/print/PrinterJob.html#pageDialog-javax.print.attribute.PrintRequestAttributeSet">https://docs.oracle.com/javase/8/docs/api/java/awt/print/PrinterJob.html#pageDialog-javax.print.attribute.PrintRequestAttributeSet</a>-<br>
      <i>[If the user cancels the dialog, the attributes will not
        reflect any changes made by the user, and the return value will
        be null.]<br>
        <br>
      </i>We were returning PageFormat instead. Fixed to return null in
      such case. <br>
      <br>
      Regards<br>
      Prasanta<br>
    </blockquote>
  </body>
</html>