On 8/13/2015 8:16 AM, shilpi rastogi wrote:
Hi all,
Please review a test bug fix
TEST : closed/javax/swing/plaf/basic/BasicHTML/4960629/bug4960629.java
BUG ID - https://bugs.openjdk.java.net/browse/JDK-7124238
Pleasemove the test from closed repo to open repo.
The webrev is: http://cr.openjdk.java.net/~sgupta/7124238/webrev.01
add to open repo
- The long lines should be split so they fit to a page
- Exceptions should be re-thrown. In other case they are just
printed but jtreg decides that a test is passed.
69 try {
70 createAndShowGUI();
71 } catch (AWTException e) {
72 e.printStackTrace();
73 }
- The test methods can throw a general Exceptioninstead of several
concrete ones. This usually makes a test more readable.
It also allows to omit try/catch block for the the
UIManager.setLookAndFeel() call.
Thanks,
Alexandr.
http://cr.openjdk.java.net/~sgupta/7124238/webrev.00/
<http://cr.openjdk.java.net/%7Ekshefov/8017187/webrev.diff/> - diff
with previous version of the closed test.
Thanks,
Shilpi
