<AWT Dev> <AWT dev>[9] Review request for JDK-8137137: [macosx] The native dialog doesn't have 'close'(X) button on Mac OS.

[Phil Race]

Hi,

Good to get rid of one applet test. Only 1,499 to go :-)

Even though you effectively re-wrote this you should set the (c) as a range "2007, 2016"

Also I think we should try to use the natural size of the component rather than explicitly
setting the size :-

instructionFrame.setBounds(0, 0, 650, 230);

instructionTextArea.setBounds(10, 10, 650, 230);

-phil.

On 04/15/2016 10:53 AM, Manajit Halder wrote:
> Hi All,
>
> Kindly review the fix for JDK9.
>
> Bug:
> https://bugs.openjdk.java.net/browse/JDK-8137137
>
> Webrev:
> http://cr.openjdk.java.net/~mhalder/8137137/webrev.00/ 
> <http://cr.openjdk.java.net/%7Emhalder/8137137/webrev.00/>
>
> Issue:
> The native print dialog and native page dialog doesn’t display 
> close(X) button in Mac OS X.
>
> Cause:
> The reported problem is the expected behaviour on Mac OS. The native 
> print dialog does not have ‘close’(X) button on MacOS, instead the 
> dialog contains OK or Cancel buttons and expects either Cancel or OK 
> button clicks from the user.
> The native print dialogs (“native print dialog” and “native page 
> dialogs”) calls Cocoa method runModalWithPrintInfo on MacOS to display 
> the print and page dialogs. runModalWithPrintInfo method does not have 
> any close button and also the normal behaviour of a modal window is to 
> exclude all other windows and panels from receiving events. Therefore 
> as long as the print or page dialog window is open, no other windows 
> will receive any events.
>
> The user is expected to either click OK button or Cancel button (or 
> press the ESC key) to close the print or page dialog.
>
> Fix:
> No fix required. The code changed for the following reasons:
> To add proper instruction with colse(X) button behaviour on Mac OS X.
>   Rewrite the test case without Applets.
>
> Regards,
> Manajit

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20160415/f7d03d34/attachment-0001.html>