On Tue, 23 Jul 2024 11:08:01 GMT, Abhishek Kumar <[email protected]> wrote:
>> This test was modified under >> [JDK-8328403](https://bugs.openjdk.org/browse/JDK-8328403) bug to remove the >> applet usage. Test modification includes the instruction change as well >> which says that `Check that there is a panel with "Text Preview Panel" text >> and with title "Preview" in the JColorChooser`. Applet test didn't say >> explicitly about the panel title. >> >> When the test is run with `GTKLookAndFeel` option, `Preview` title is >> missing due to the `showPreviewPanelText` property defined for >> `GTKLookAndFeel` is set to **FALSE**. `showPreviewPanelText` property is not >> defined for other L&Fs and that leads to fallback to the default value of >> **TRUE** for the property while lookup. >> >> Proposed fix is to change the default value of `showPreviewPanelText` >> property to **TRUE** for GTK L&F. > > Abhishek Kumar has updated the pull request incrementally with one additional > commit since the last revision: > > removed bug id from jtreg tag test/jdk/javax/swing/JColorChooser/Test6977726.java line 46: > 44: Test passes if the panel is as described, test fails > otherwise. > 45: > 46: Note: "Preview" title is not applicable for GTK Look and > Feel."""; The test summary says it tests `setPreviewPanel`...if it's not present in native, then the full test is not applicable for GTK..I guess you can check for UIManager.getLookAndFeel() is GTK, then bail out.. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/20104#discussion_r1687902984
