RFR: JDK-8353755 : Add a helper method to Util - findComponent() [v5]

Harshitha Onkar honkar at openjdk.org
Wed Nov 5 18:02:29 UTC 2025


On Tue, 4 Nov 2025 14:01:16 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> I also suggest calling `findComponent` instead of `findComponentImpl` — then you can remove the line *“Always run this method on the EDT thread”* from the javadoc comment.
>> 
>> Also, change “a sub component” to “a subcomponent”.
>
>> It fails because the code I proposed is plainly wrong.
>> 
>> ```java
>>     public static Component findSubComponent(Component parent, String className) {
>>         return findComponentImpl((Container) parent,
>>                                  c -> c.getClass().getName().contains(className));
>>     }
>> ```
> 
> This time I tested the above code by running the following tests `test/jdk/javax/swing/JFileChooser/FileChooserToolTipTest.java test/jdk/javax/swing/JFileChooser/FCPopupMenuOrientationTest.java` which use `Util.findSubComponent` and the tests passed for me.

Updated.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27944#discussion_r2495585933


More information about the client-libs-dev mailing list