Integrated: 8281745: Create a regression test for JDK-4514331

2022-02-21 Thread Manukumar V S
On Tue, 15 Feb 2022 02:48:54 GMT, Manukumar V S wrote: > Create a regression test for JDK-4514331. > It checks whether pressing 'Tab' key always shift focus to next component, > even though the current focus is in JTextArea and some text is already > selected. > > Testing: > I have verified th

Re: RFR: 8281738: Create a regression test for checking the 'Space' key activation of focused Button [v6]

2022-02-21 Thread Manukumar V S
> This test verifies in all platforms and in all 'Look and Feels' that pressing > the 'Space' key generates ActionEvent on focused Button or not. Manukumar V S has updated the pull request incrementally with one additional commit since the last revision: Fixed review comments: Updated code co

Re: RFR: 8281988: Create a regression test for JDK-4618767 [v4]

2022-02-21 Thread Manukumar V S
> Create a regression test for > [JDK-4618767](https://bugs.openjdk.java.net/browse/JDK-4618767) > > Issue identified in > [JDK-4618767](https://bugs.openjdk.java.net/browse/JDK-4618767): > Typing a letter while a JList has focus now makes the selection jump to the > first/next node/item whose

Re: RFR: 8280964: [Linux aarch64] : drawImage dithers TYPE_BYTE_INDEXED images incorrectly

2022-02-21 Thread Dmitry Markov
On Mon, 21 Feb 2022 18:13:12 GMT, Phil Race wrote: > As per the bug evaluation, the root of this issue is that on this platform > "char" is treated as "unsigned char" > and the code here expects it to be "signed char", so we need to make it > explicitly so. > For all other supported platforms t

Re: RFR: 8280964: [Linux aarch64] : drawImage dithers TYPE_BYTE_INDEXED images incorrectly

2022-02-21 Thread Sergey Bylokhov
On Mon, 21 Feb 2022 18:13:12 GMT, Phil Race wrote: > As per the bug evaluation, the root of this issue is that on this platform > "char" is treated as "unsigned char" > and the code here expects it to be "signed char", so we need to make it > explicitly so. > For all other supported platforms t

Re: RFR: 8260328: Drop redundant CSS properties from java.desktop HTML files

2022-02-21 Thread Sergey Bylokhov
On Sun, 20 Feb 2022 17:04:39 GMT, Alexey Ivanov wrote: > Drop redundant `text-align: left` from `` elements, which don't affect > the table alignment. Then drop `text-align: center` from ` elements which > counter-act the align set on the ``. > > Also drop `vertical-align: top` from table cell

Re: RFR: 8236907: JTable added to nested panels does not paint last visible row [v6]

2022-02-21 Thread Tejesh R
> Issue in painting last row of JTable due to decrement in `MaxRow `during > computation. Differences in `MaxRow `computation between `TablePrintable` and > `BasicTableUI `class. The Bug is fixed in both the classes. Test Case added > to validate the same. Tejesh R has updated the pull request

Re: RFR: 8281745: Create a regression test for JDK-4514331 [v4]

2022-02-21 Thread Sergey Bylokhov
On Sat, 19 Feb 2022 07:55:26 GMT, Manukumar V S wrote: >> Create a regression test for JDK-4514331. >> It checks whether pressing 'Tab' key always shift focus to next component, >> even though the current focus is in JTextArea and some text is already >> selected. >> >> Testing: >> I have veri

Re: RFR: 8277922: Unable to click JCheckBox in JTable through Java Access Bridge

2022-02-21 Thread Sergey Bylokhov
On Sat, 12 Feb 2022 00:52:34 GMT, Anton Litvinov wrote: >> src/java.desktop/share/classes/javax/swing/JTable.java line 5508: >> >>> 5506: (table != null) && table.isEnabled() && >>> 5507: table.isCellEditable(row, column)) { >>> 5508:

Re: RFR: JDK-8282046: Create a regression test for CCC8000326

2022-02-21 Thread Alexey Ivanov
On Thu, 17 Feb 2022 09:56:45 GMT, Srinivas Mandalika wrote: > Create a regression test for CCC8000326 > > Issue is identified by > [JDK-8000326](https://bugs.openjdk.java.net/browse/JDK-8000326), which > identifies that after focus moves into JMenuBar, whose focus traversal key is > disabled

Re: RFR: 8281569: Create tests for Frame.setMinimumSize() method

2022-02-21 Thread Alexey Ivanov
On Fri, 11 Feb 2022 17:28:09 GMT, Srinivas Mandalika wrote: > Create tests for setMinimumSize functionality > Create a test which verifies that on increasing the Frame's minimumSize, it > gets reflected in the subsequent getSize call > Create a test which verifies that on frame resizes back to

Re: RFR: 8281738: Create a regression test for checking the 'Space' key activation of focused Button [v5]

2022-02-21 Thread Alexey Ivanov
On Sat, 19 Feb 2022 07:47:34 GMT, Manukumar V S wrote: >> This test verifies in all platforms and in all 'Look and Feels' that >> pressing the 'Space' key generates ActionEvent on focused Button or not. > > Manukumar V S has updated the pull request incrementally with one additional > commit si

Re: RFR: 8281988: Create a regression test for JDK-4618767 [v3]

2022-02-21 Thread Alexey Ivanov
On Sat, 19 Feb 2022 08:09:32 GMT, Manukumar V S wrote: >> Create a regression test for >> [JDK-4618767](https://bugs.openjdk.java.net/browse/JDK-4618767) >> >> Issue identified in >> [JDK-4618767](https://bugs.openjdk.java.net/browse/JDK-4618767): >> Typing a letter while a JList has focus now

Re: RFR: 8281988: Create a regression test for JDK-4618767 [v3]

2022-02-21 Thread Alexey Ivanov
On Sat, 19 Feb 2022 08:09:32 GMT, Manukumar V S wrote: >> Create a regression test for >> [JDK-4618767](https://bugs.openjdk.java.net/browse/JDK-4618767) >> >> Issue identified in >> [JDK-4618767](https://bugs.openjdk.java.net/browse/JDK-4618767): >> Typing a letter while a JList has focus now

Re: RFR: 8280035: Use Class.isInstance instead of Class.isAssignableFrom where applicable

2022-02-21 Thread Peter Levart
On Mon, 17 Jan 2022 08:28:35 GMT, Erik Gahlin wrote: >> src/jdk.jfr/share/classes/jdk/jfr/internal/TypeLibrary.java line 405: >> >>> 403: Object res = m.invoke(a, new Object[0]); >>> 404: if (res instanceof Annotation[]) { >>> 405:

RFR: 8280964: [Linux aarch64] : drawImage dithers TYPE_BYTE_INDEXED images incorrectly

2022-02-21 Thread Phil Race
As per the bug evaluation, the root of this issue is that on this platform "char" is treated as "unsigned char" and the code here expects it to be "signed char", so we need to make it explicitly so. For all other supported platforms this is a no-op since that is the default elsewhere. So this sh

Integrated: 8281523: Accessibility: Conversion from string literal loses const qualifier

2022-02-21 Thread Daniel Jeliński
On Tue, 8 Feb 2022 11:41:16 GMT, Daniel Jeliński wrote: > This patch fixes jdk.accessibility compilation under VisualStudio 2019 with > /Zc:strictStrings enabled. > > Strict string handling is already enabled in GCC and Clang. With some effort > we should be able to enable it in MSVC as well.

Re: RFR: 8281523: Accessibility: Conversion from string literal loses const qualifier [v2]

2022-02-21 Thread Daniel Jeliński
On Mon, 14 Feb 2022 21:25:42 GMT, Daniel Jeliński wrote: >> This patch fixes jdk.accessibility compilation under VisualStudio 2019 with >> /Zc:strictStrings enabled. >> >> Strict string handling is already enabled in GCC and Clang. With some effort >> we should be able to enable it in MSVC as

Re: RFR: 8281523: Accessibility: Conversion from string literal loses const qualifier [v2]

2022-02-21 Thread Alexander Zuev
On Mon, 14 Feb 2022 21:25:42 GMT, Daniel Jeliński wrote: >> This patch fixes jdk.accessibility compilation under VisualStudio 2019 with >> /Zc:strictStrings enabled. >> >> Strict string handling is already enabled in GCC and Clang. With some effort >> we should be able to enable it in MSVC as

Re: CFV: New Client Libraries Group Member : Dmitry Markov (dmarkov)

2022-02-21 Thread Alexander Zuev
Vote: yes On 2/9/22 12:06, Philip Race wrote: I hereby nominate Dmitry Markov (dmarkov) to Membership in the Client Libraries Group. Dmitry has worked in the client libraries sustaining team at Oracle since 2013 and in that time has contributed 61 client fixes [1], [2]. He has been active on

Re: CFV: New Client Libraries Group Member : Alexey Ivanov (aivanov)

2022-02-21 Thread Alexander Zuev
Vote: yes On 2/9/22 12:06, Philip Race wrote: I hereby nominate Alexey Ivanov (aivanov) to Membership in the Client Libraries Group. Alexey has worked in the client libraries sustaining team at Oracle since 2014 and in that time has contributed 60 client fixes [1], [2]. He has been active on

Re: RFR: 8236907: JTable added to nested panels does not paint last visible row [v5]

2022-02-21 Thread Prasanta Sadhukhan
On Mon, 21 Feb 2022 10:03:30 GMT, Tejesh R wrote: >> Issue in painting last row of JTable due to decrement in `MaxRow `during >> computation. Differences in `MaxRow `computation between `TablePrintable` >> and `BasicTableUI `class. The Bug is fixed in both the classes. Test Case >> added to va

Re: RFR: 8236907: JTable added to nested panels does not paint last visible row [v4]

2022-02-21 Thread Tejesh R
On Mon, 21 Feb 2022 05:50:06 GMT, Prasanta Sadhukhan wrote: >> Tejesh R has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 8236907: JTable added to nested panels does not paint last > > test/jdk/javax/swing/JTable/8236907/LastVisibleRow.ja

Re: RFR: 8276849: Refresh the window icon on graphics configuration changes [v3]

2022-02-21 Thread Alexey Ivanov
On Mon, 21 Feb 2022 07:57:32 GMT, Emmanuel Bourg wrote: >> When a list of icons is set on a window, the most appropiate icon is >> selected depending on the graphics configuration. But if the graphics >> configuration changes (because the window is moved to a different screen, or >> because th

Re: RFR: 8276849: Refresh the window icon on graphics configuration changes [v3]

2022-02-21 Thread Alexey Ivanov
On Mon, 21 Feb 2022 07:57:32 GMT, Emmanuel Bourg wrote: >> When a list of icons is set on a window, the most appropiate icon is >> selected depending on the graphics configuration. But if the graphics >> configuration changes (because the window is moved to a different screen, or >> because th

Re: RFR: 8276849: Refresh the window icon on graphics configuration changes [v3]

2022-02-21 Thread Alexey Ivanov
On Mon, 21 Feb 2022 12:53:35 GMT, Alexey Ivanov wrote: >> Emmanuel Bourg has updated the pull request incrementally with five >> additional commits since the last revision: >> >> - Test case for the window icon update on DPI change (other changes) >> - Test case for the window icon update on

Re: RFR: 8280035: Use Class.isInstance instead of Class.isAssignableFrom where applicable

2022-02-21 Thread Andrey Turbanov
On Thu, 20 Jan 2022 01:34:20 GMT, Phil Race wrote: >In short I see insufficient value in the changes here and would prefer you >drop the client part so I don't have to worry about it. I think, usage of `isInstance` is much clear for most java developers. Everyone knows about java _instanceof_

Re: RFR: 8280035: Use Class.isInstance instead of Class.isAssignableFrom where applicable

2022-02-21 Thread Andrey Turbanov
On Thu, 13 Jan 2022 08:25:22 GMT, Andrey Turbanov wrote: > Method `Class.isAssignableFrom` is often used in form of: > > if (clazz.isAssignableFrom(obj.getClass())) { > Such condition could be simplified to more shorter and performarnt code > > if (clazz.isInstance(obj)) { > > Repl

Integrated: 8280861: Robot color picker broken on Linux with scaling above 100%

2022-02-21 Thread Maxim Kartashev
On Thu, 10 Feb 2022 14:08:48 GMT, Maxim Kartashev wrote: > The primary API for image capture on modern Linuxes is > `gdk_pixbuf_get_from_window()` that expects both coordinates and the size > unscaled (i.e. not multiplied by the current desktop scale). At the same > time, `gtk3_interface.c:gtk

Integrated: 8282147: [TESTBUG] waitForIdle after creating frame in JSpinnerMouseAndKeyPressTest.java

2022-02-21 Thread Manukumar V S
On Sun, 20 Feb 2022 08:01:28 GMT, Manukumar V S wrote: > Call Robot.waitForIdle() after createUI and before getting the location of > the JSpinner component in the test: > javax/swing/JSpinner/4515999/JSpinnerMouseAndKeyPressTest.java. > > This is needed to ensure all the events for creating a

Re: RFR: 8236907: JTable added to nested panels does not paint last visible row [v5]

2022-02-21 Thread Tejesh R
> Issue in painting last row of JTable due to decrement in `MaxRow `during > computation. Differences in `MaxRow `computation between `TablePrintable` and > `BasicTableUI `class. The Bug is fixed in both the classes. Test Case added > to validate the same. Tejesh R has updated the pull request

Re: RFR: 8282147: [TESTBUG] waitForIdle after creating frame in JSpinnerMouseAndKeyPressTest.java

2022-02-21 Thread Alexey Ivanov
On Sun, 20 Feb 2022 08:01:28 GMT, Manukumar V S wrote: > Call Robot.waitForIdle() after createUI and before getting the location of > the JSpinner component in the test: > javax/swing/JSpinner/4515999/JSpinnerMouseAndKeyPressTest.java. > > This is needed to ensure all the events for creating a

Re: RFR: 8241192: [macosx] Wrong letter typed after ´ symbol when using Finnish layout

2022-02-21 Thread Nikita Provotorov
On Fri, 28 Jan 2022 09:38:35 GMT, Prasanta Sadhukhan wrote: > If we press " ´ " (on U.S. keyboard layout it is the [+] button just before > the [delete] button), and then press "s" , 2 "´" characters are entered in a > text field instead of "´s" with "Finish" keyboard layout. > This is because

Re: RFR: 8037965: NullPointerException in TextLayout.getBaselineFromGraphic() for JTextComponents

2022-02-21 Thread Alexey Ivanov
On Wed, 16 Feb 2022 07:44:27 GMT, Prasanta Sadhukhan wrote: > When invalid unicode codepoints 0x and 0x1 are added to an empty > javax.swing.JTextComponent (JTextArea or JTextField), the view cannot be > updated due to a NullPointerException in TextLayout.getBaselineFromGraphic() > ow

Re: RFR: 8281523: Accessibility: Conversion from string literal loses const qualifier [v2]

2022-02-21 Thread Alexey Ivanov
On Mon, 14 Feb 2022 21:25:42 GMT, Daniel Jeliński wrote: >> This patch fixes jdk.accessibility compilation under VisualStudio 2019 with >> /Zc:strictStrings enabled. >> >> Strict string handling is already enabled in GCC and Clang. With some effort >> we should be able to enable it in MSVC as

Re: RFR: 8185261: Font fallback sometimes doesn't work in Swing text components [v2]

2022-02-21 Thread Dmitry Batrak
On Sun, 6 Feb 2022 19:53:45 GMT, Phil Race wrote: >> Dmitry Batrak has updated the pull request incrementally with one additional >> commit since the last revision: >> >> remove 'headful' requirement from test case > >> As you say, one could possibly also try to compare font2DHandle-s instead

Re: CFV: New Client Libraries Group Member : Dmitry Markov (dmarkov)

2022-02-21 Thread Jayathirth D V
Vote : Yes Thanks, Jay > On 10-Feb-2022, at 1:36 AM, Philip Race wrote: > > I hereby nominate Dmitry Markov (dmarkov) to Membership in the Client > Libraries Group. > > Dmitry has worked in the client libraries sustaining team at Oracle since > 2013 and in that time > has contributed 61 clie

Re: CFV: New Client Libraries Group Member : Alexey Ivanov (aivanov)

2022-02-21 Thread Jayathirth D V
Vote : Yes Thanks, Jay > On 10-Feb-2022, at 1:36 AM, Philip Race wrote: > > I hereby nominate Alexey Ivanov (aivanov) to Membership in the Client > Libraries Group. > > Alexey has worked in the client libraries sustaining team at Oracle since > 2014 and in that time > has contributed 60 clie