Re: RFR: 8328553: Get rid of JApplet in test/jdk/sanity/client/lib/SwingSet2/src/DemoModule.java

[Harshitha Onkar]

On Sat, 3 Aug 2024 18:07:16 GMT, lawrence.andrews <d...@openjdk.org> wrote:

> 1) Replaced JApplet with JPanel 
> 2) Tests passed before and after fix

Changes look good.

Not related to the fix, but when you click on File > About, following exception 
is seen due to missing image (resources/images/About.jpg). Maybe this option is 
not required for the sanity test in which case it can be disabled.

![image](https://github.com/user-attachments/assets/04307a44-4fd9-41bd-bac6-5f8f7ae7a293)

About.jpg is available jfc/SwingSet2/ demo version.


Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException: Cannot 
invoke "java.net.URL.toExternalForm()" because "location" is null
        at java.desktop/javax.swing.ImageIcon.<init>(ImageIcon.java:232)
        at SwingSet2.createImageIcon(SwingSet2.java:783)
        at SwingSet2$AboutPanel.<init>(SwingSet2.java:1161)

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

PR Comment: https://git.openjdk.org/jdk/pull/20450#issuecomment-2271904762