On Wed, 11 Jun 2025 19:18:15 GMT, Phil Race <p...@openjdk.org> wrote:
>> This is the implementation of JEP 504 - Remove the Applet API.
>> API changes are
>> - Remove the entire java.applet package
>> - Remove the javax/swing/JApplet class
>> - Remove applet related APIs in java.beans
>> - Update javadoc referring to applets, including one gif image - now changed
>> to an svg image
>> -
>> Other changes are
>> - Remove references to the removed classes
>> - Remove obsolete tests
>> - Update obsolete code comments
>>
>> sun.awt.AppContext is even more obsolete now than it was before, but
>> eliminating uses of that will be is not required,
>> and will be follow-on internal clean up, at a later date, under unrelated
>> bug ids, and likely not completed in the same
>> release as this JEP is integrated.
>>
>> I have extensively tested this - running all the automated tests used by CI
>> tiers 1 to 8.
>
> Phil Race has updated the pull request incrementally with one additional
> commit since the last revision:
>
> 8359053
src/java.desktop/share/classes/javax/imageio/spi/IIORegistry.java line 113:
> 111: * registry functions.
> 112: *
> 113: * <p> Each {@code ThreadGroup} will receive its own instance.
Note that this new specification is not tied to AppContext. Do we still want to
support ThreadGroup sandboxing even if AppContext will be removed? Can we drop
it now, since current spec clearly tied to applets and AppContext usage?
src/java.desktop/share/classes/javax/swing/BufferStrategyPaintManager.java line
796:
> 794: BufferStrategy bs = null;
> 795: try {
> 796: ((Window)root).createBufferStrategy(2, caps);
We probably can change the type of "root" to "Window".
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25698#discussion_r2140913064
PR Review Comment: https://git.openjdk.org/jdk/pull/25698#discussion_r2140914921
