On Thu, 12 Jun 2025 21:54:03 GMT, Phil Race <p...@openjdk.org> wrote:

>> src/java.desktop/share/classes/java/awt/Window.java line 3864:
>> 
>>> 3862:                 content.setOpaque(isOpaque);
>>> 3863: 
>>> 3864:                 // Iterate down one level to see whether we have (eg) 
>>> a JInternalFrame
>> 
>> I think it is more related to top level windows like JWindow/JDialog etc.
>
> I am not sure that's the case. I think it best to leave the comment as it is 
> - minus mention of applet.

After the applet removal, this could be dead code. It might be better not to 
mention JInternalFrame unless we have prove that the code actually works for 
it, add a TODO? or something?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25698#discussion_r2143797148

Reply via email to