RFR: 6450193: After the first Serialization, JTableHeader does not uninstall its UI [v3]
Alexey Ivanov
aivanov at openjdk.org
Mon Sep 25 11:24:14 UTC 2023
On Mon, 25 Sep 2023 06:53:49 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
>> test/jdk/javax/swing/JTableHeader/SerializeJTableHeader.java line 55:
>>
>>> 53: for (UIManager.LookAndFeelInfo laf : UIManager.getInstalledLookAndFeels()) {
>>> 54: System.out.println("Testing L&F: " + laf);
>>> 55: SwingUtilities.invokeAndWait(() -> setLookAndFeel(laf));
>>
>> Why did you choose to set LAF on EDT? You then continue with creating and serialising the component on the main thread.
>
> I was not sure if any event processing needs to be done for lookandfeel change particularly native l&f like GTK and Aqua.
> Seems like it works with both EDT and non-EDT...tried with repeat count 50....so removed EDT usage...
> however GTK is not supported on headless environment so made the test run on headful environment.
I never thought about it… Yet if there's anything that depends on access to native platform, it probably needs to run on the toolkit thread.
I'm for keeping it simpler: if it works correctly on the main thread here, let's keep it this way.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15507#discussion_r1335747606
More information about the client-libs-dev
mailing list