RFR: 8328238: Convert few closed manual applet tests to main [v4]

Phil Race prr at openjdk.org
Wed Mar 20 23:11:23 UTC 2024


On Wed, 20 Mar 2024 16:46:45 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> Few closed manual applet tests are converted to main based and opensourced
>
> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
> 
>   REview update

Marked as reviewed by prr (Reviewer).

test/jdk/javax/swing/JFrame/bug4419914.java line 71:

> 69:         frame.enableInputMethods(false);
> 70:         frame.getContentPane().setComponentOrientation(ComponentOrientation.RIGHT_TO_LEFT);
> 71:         frame.getContentPane().setLocale(Locale.ENGLISH);

The requirement to use getContentPane() has been obsolete since JDK 1.5 ...
here's an external page about it https://coderanch.com/t/345686/java/getContentPane
But I don't generally tell people to delete except here there are 9 calls cluttering up this code.
So not critical, but a suggestion.

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

PR Review: https://git.openjdk.org/jdk/pull/18390#pullrequestreview-1950401141
PR Review Comment: https://git.openjdk.org/jdk/pull/18390#discussion_r1533017080


More information about the client-libs-dev mailing list