RFR: 8274032 : Remove jtreg tag manual=yesno for java/awt/print/PrinterJob/ImagePrinting/ImageTypes.java & show test UI

[Pankaj Bansal]

On Tue, 21 Sep 2021 03:24:24 GMT, lawrence.andrews <github.com+87324768+lawrence-andrew at openjdk.org> wrote:

> Problem : 
> 1) Testcase was failing due to parse exception yesno
> 2) After removing yesno tag test frame and immediately get disposed
> 
> Fix : 
> 1) Removed yesno tag
> 2) Show the test frame with pass, fail button.  Now user can click on the `print' button to show the print dialog and take the print out and compare the print with the graphics rendered on the frame. Click either on the 'Pass' or 'Fail' button to make the pass or fail also clicking the 'Pass' or 'Fail' closes the test frame.
> 3) Added the timeout If not interaction is performed on the test frame.
> 
> @shurymury

test/jdk/java/awt/print/PrinterJob/ImagePrinting/ImageTypes.java line 83:

> 81: import static java.awt.image.BufferedImage.TYPE_INT_RGB;
> 82: import static java.awt.image.BufferedImage.TYPE_USHORT_555_RGB;
> 83: import static java.awt.image.BufferedImage.TYPE_USHORT_GRAY;

Thats lot of static imports from just two classes. This just looks weird. Is it recommended practice to do? I am not saying this is wrong as you are removing wild card imports, but it just does not look good to have so many statement for importing from two classes.

-------------

PR: https://git.openjdk.java.net/jdk/pull/5595