RFR: 8322135: javax/swing/JTable/JTableScrollPrintTest.java & javax/swing/JTable/PrintAllPagesTest.java throws java.lang.InternalError: HTHEME is null [v4]

[Alexey Ivanov]

On Tue, 23 Apr 2024 10:30:49 GMT, Tejesh R <tr at openjdk.org> wrote:

>> src/java.desktop/windows/classes/sun/awt/windows/ThemeReader.java line 110:
>> 
>>> 108:            // See documentation for SetWindowTheme on MSDN.
>>> 109:            setWindowTheme(widget.substring(0, i));
>>> 110:            theme = getOpenThemeValue(widget.substring(i + 2), dpi);
>> 
>> I suggest changing the type of `theme` variable from `Long` to `long` as well as the return value of `openThemeImpl`.
>
> Means only till openThemeImpl return value and not further up the hierarchy ?

Yes, only the low-level methods until it's put into a map.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18706#discussion_r1576031141