8305352: updateIconImages may lead to deadlock after JDK-8276849

Dmitry Cherepanov · Dmitry Cherepanov · commit f968da97a5a5 · 2023-04-25T06:56:32.000Z
Reviewed-by: aivanov, serb
diff --git a/src/java.desktop/windows/classes/sun/awt/windows/WWindowPeer.java b/src/java.desktop/windows/classes/sun/awt/windows/WWindowPeer.java
@@ -600,7 +600,9 @@ public void updateGC() {
             newDev.addDisplayChangedListener(this);
         }
 
-        updateIconImages();
+        if (((Window)target).isVisible()) {
+            updateIconImages();
+        }
 
         AWTAccessor.getComponentAccessor().
             setGraphicsConfiguration((Component)target, winGraphicsConfig);

Original file line number	Diff line number	Diff line change
`@@ -600,7 +600,9 @@ public void updateGC() {`
`600`	`600`	`newDev.addDisplayChangedListener(this);`
`601`	`601`	`}`
`602`	`602`
`603`		`- updateIconImages();`
	`603`	`+ if (((Window)target).isVisible()) {`
	`604`	`+ updateIconImages();`
	`605`	`+ }`
`604`	`606`
`605`	`607`	`AWTAccessor.getComponentAccessor().`
`606`	`608`	`setGraphicsConfiguration((Component)target, winGraphicsConfig);`