Re: RFR: 8224968: javax/swing/JColorChooser/Test6827032.java times out in Windows 10

2025-02-28 Thread Alexander Zvegintsev
On Fri, 28 Feb 2025 15:42:51 GMT, Alexey Ivanov wrote: > Thread dump shows no AWT threads, yet jtreg harness still waits for the test > thread to finish, in particular it waits for > [StreamCopier](https://github.com/openjdk/jtreg/blob/759946dedbafa423552851ecb98bc3bb8dcf30ec/src/share/classes/

Re: RFR: 8344892: beans/finder/MethodFinder.findMethod incorrectly returns null

2025-02-28 Thread Alexander Zvegintsev
On Fri, 28 Feb 2025 14:41:41 GMT, Alexander Zvegintsev wrote: > During the [JDK-8344891 Remove uses of sun.misc.ReflectUtil in > java.desktop](https://bugs.openjdk.org/browse/JDK-8344891) it was discovered > that `beans/finder/MethodFinder.findMethod' incorrectly retu

RFR: 8344892: beans/finder/MethodFinder.findMethod incorrectly returns null

2025-02-28 Thread Alexander Zvegintsev
During the [JDK-8344891 Remove uses of sun.misc.ReflectUtil in java.desktop](https://bugs.openjdk.org/browse/JDK-8344891) it was discovered that `beans/finder/MethodFinder.findMethod' incorrectly returned null if a signature was not in the cache and added it to the cache if it was already there

Re: RFR: 8348110: Update LCMS to 2.17 [v2]

2025-02-27 Thread Alexander Zvegintsev
On Thu, 27 Feb 2025 19:39:37 GMT, Alisen Chung wrote: >> upgrading third party library to newest version > > Alisen Chung has updated the pull request incrementally with one additional > commit since the last revision: > > readd license header to files src/java.desktop/share/native/liblcms/L

Re: RFR: 8350813: Rendering of bulky sound bank from MIDI sequence can cause OutOfMemoryError

2025-02-26 Thread Alexander Zvegintsev
On Wed, 26 Feb 2025 22:26:48 GMT, Alexander Zuev wrote: > - Check that the calculated audio data size does not exceed available heap > memory before committing to the rendering > - Add a test case test/jdk/javax/sound/midi/BulkSoundBank/BulkSoundBank.java line 33: > 31: * @bug 8350813 > 32:

Re: RFR: 8280991: [XWayland] No displayChanged event after setDisplayMode call [v2]

2025-02-26 Thread Alexander Zvegintsev
On Thu, 27 Feb 2025 00:56:23 GMT, Harshitha Onkar wrote: >> Alexander Zvegintsev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> NoResizeEventOnDMChangeTest update > > test/jdk/java/awt/Fu

Re: RFR: 8280991: [XWayland] No displayChanged event after setDisplayMode call [v2]

2025-02-25 Thread Alexander Zvegintsev
On Tue, 25 Feb 2025 15:57:38 GMT, Mikhail Yankelevich wrote: >> Alexander Zvegintsev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> NoResizeEventOnDMChangeTest update > > test/jdk/java/awt/FullScre

Re: RFR: 8280991: [XWayland] No displayChanged event after setDisplayMode call [v2]

2025-02-25 Thread Alexander Zvegintsev
excluded for versions below that. > > --- > > Testing looks good (manual + automated). Alexander Zvegintsev has updated the pull request incrementally with one additional commit since the last revision: NoResizeEventOnDMChangeTest update - Changes: - all: https://git.

RFR: 8280991: [XWayland] No displayChanged event after setDisplayMode call

2025-02-25 Thread Alexander Zvegintsev
Wayland clients are by design not allowed to change the resolution in Wayland. XRandR in Xwayland is just an emulation, it doesn't actually change the desktop resolution. This emulation is per window/x11 client, so different clients can have different emulated resolutions at the same time. Any r

Re: RFR: 8344981: [REDO] JDK-6672644 JComboBox still scrolling if switch to another window and return back [v7]

2025-02-24 Thread Alexander Zvegintsev
On Mon, 24 Feb 2025 21:44:12 GMT, Damon Nguyen wrote: >> Redo for JComboBox infinite scrolling issue. The issue is that when a >> scrollbar is clicked and held, if the user switches focus (ex: ALT+TAB) >> while scrolling, when focused is returned to the scrolling application, the >> JComboBox

Re: RFR: 8150564: Migrate useful ExtendedRobot methods into awt.Robot [v15]

2025-02-24 Thread Alexander Zvegintsev
On Thu, 6 Feb 2025 22:03:54 GMT, Alisen Chung wrote: >> Some useful methods in ExtendedRobot should be migrated into Robot itself so >> that ExtendedRobot can be removed in the future. The tests using >> ExtendedRobot for these migrated methods are changed to use only Robot >> (removing unnece

Re: RFR: 8344981: [REDO] JDK-6672644 JComboBox still scrolling if switch to another window and return back [v6]

2025-02-21 Thread Alexander Zvegintsev
On Wed, 19 Feb 2025 23:04:39 GMT, Damon Nguyen wrote: >> Redo for JComboBox infinite scrolling issue. The issue is that when a >> scrollbar is clicked and held, if the user switches focus (ex: ALT+TAB) >> while scrolling, when focused is returned to the scrolling application, the >> JComboBox

Re: RFR: 8348106: Catch C++ exception in Java_sun_awt_windows_WTaskbarPeer_setOverlayIcon [v5]

2025-02-21 Thread Alexander Zvegintsev
On Fri, 21 Feb 2025 16:29:08 GMT, Rajat Mahajan wrote: >> **Issue:** >> The JNI method `Java_sun_awt_windows_WTaskbarPeer_setOverlayIcon `calls >> `CreateIconFromRaster `that can throw a C++ exception. >> >> The C++ exception must be caught and must not be allowed to escape the JNI >> method.

Re: RFR: 8348596: Update FreeType to 2.13.3 [v3]

2025-02-19 Thread Alexander Zvegintsev
On Wed, 19 Feb 2025 23:29:44 GMT, Alisen Chung wrote: >> Upgrading freetype 3rd party tool to newest update > > Alisen Chung has updated the pull request incrementally with one additional > commit since the last revision: > > remove ws tool I assume testing looks good. - Marked

Re: RFR: 8350224: Deproblemlist javax/swing/JComboBox/TestComboBoxComponentRendering.java

2025-02-18 Thread Alexander Zvegintsev
On Tue, 18 Feb 2025 07:45:06 GMT, Prasanta Sadhukhan wrote: > Test is failing in ubuntu 23.x and beyond with expected red pixel not being > picked up > Increase in fontsize increase the possibility of font red pixel being picked > up. > Test is now passing in ubuntu 24.04 and in CI.. Marked

Re: RFR: 8350260: Improve HTML instruction formatting in PassFailJFrame

2025-02-18 Thread Alexander Zvegintsev
On Tue, 18 Feb 2025 11:48:09 GMT, Alexey Ivanov wrote: > **Problem:** > > When instructions are long, the formatting in `PassFailJFrame` looks off: > > 1. When the instructions are displayed on the screen for the first time, the > HTML is scrolled to the bottom, which isn't convenient; > 2. Nu

Integrated: 8349751: AIX build failure after upgrade pipewire to 1.3.81

2025-02-14 Thread Alexander Zvegintsev
On Mon, 10 Feb 2025 19:07:49 GMT, Alexander Zvegintsev wrote: > Filed as a separate issue to keep the #23426 clean. > > Fix is the same as in the > `src/java.desktop/unix/native/libpipewire/include/spa/param/audio/raw.h` part > of the [JDK-8309703](https://bugs.openjdk.org/bro

Re: RFR: 8349751: AIX build failure after upgrade pipewire to 1.3.81 [v4]

2025-02-13 Thread Alexander Zvegintsev
> Filed as a separate issue to keep the #23426 clean. > > Fix is the same as in the > `src/java.desktop/unix/native/libpipewire/include/spa/param/audio/raw.h` part > of the [JDK-8309703](https://bugs.openjdk.org/browse/JDK-8309703) Alexander Zvegintsev has updated the pull req

Integrated: 8348600: Update PipeWire to 1.3.81

2025-02-13 Thread Alexander Zvegintsev
On Mon, 3 Feb 2025 18:50:41 GMT, Alexander Zvegintsev wrote: > This changeset updates the pipewire headers to the latest available version. > It contains the minimum set of required header files needed to build the JDK. > > The updated headers are a direct copy from

Re: RFR: 8347019: Test javax/swing/JRadioButton/8033699/bug8033699.java still fails: Focus is not on Radio Button Single as Expected

2025-02-11 Thread Alexander Zvegintsev
On Tue, 11 Feb 2025 04:15:49 GMT, Prasanta Sadhukhan wrote: > Test seems to fail in ubuntu 22.04 system..It seems removing setAutoDelay is > making the test more stable in such systems. > Tested in all platforms where it is still ok without auto delay. CI job link > in JBS.. Marked as reviewe

Re: RFR: 8294155: Exception thrown before awaitAndCheck hangs PassFailJFrame

2025-02-11 Thread Alexander Zvegintsev
On Tue, 11 Feb 2025 16:17:13 GMT, Alexey Ivanov wrote: > If a manual test throws an exception during construction of `PassFailJFrame`, > the test execution hangs. When the builder pattern is used, no UI appears on > the screens, but the Java process does not terminate automatically because > t

Re: RFR: 8348600: Update PipeWire to 1.3.81

2025-02-11 Thread Alexander Zvegintsev
On Tue, 11 Feb 2025 01:07:33 GMT, Harshitha Onkar wrote: > One thing that I noticed and wanted discuss further: This is when I delete > the screencast-tokens.properties file and the permission is granted for the > first screen share (new token generated), it asks for permission every time > fo

Re: RFR: 8349751: AIX build failure after upgrade pipewire to 1.3.81 [v3]

2025-02-11 Thread Alexander Zvegintsev
On Tue, 11 Feb 2025 08:33:09 GMT, Julian Waters wrote: >> Alexander Zvegintsev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> move fix to spa/utils/endian.h > > src/java.desktop/unix/native/libpipewire

Re: RFR: 8349751: AIX build failure after upgrade pipewire to 1.3.81 [v3]

2025-02-11 Thread Alexander Zvegintsev
> Filed as a separate issue to keep the #23426 clean. > > Fix is the same as in the > `src/java.desktop/unix/native/libpipewire/include/spa/param/audio/raw.h` part > of the [JDK-8309703](https://bugs.openjdk.org/browse/JDK-8309703) Alexander Zvegintsev has updated t

Re: RFR: 8349751: AIX build failure after upgrade pipewire to 1.3.81 [v2]

2025-02-11 Thread Alexander Zvegintsev
> Filed as a separate issue to keep the #23426 clean. > > Fix is the same as in the > `src/java.desktop/unix/native/libpipewire/include/spa/param/audio/raw.h` part > of the [JDK-8309703](https://bugs.openjdk.org/browse/JDK-8309703) Alexander Zvegintsev has updated t

Re: RFR: 8344981: [REDO] JDK-6672644 JComboBox still scrolling if switch to another window and return back

2025-02-10 Thread Alexander Zvegintsev
On Tue, 4 Feb 2025 23:35:32 GMT, Damon Nguyen wrote: > Redo for JComboBox infinite scrolling issue. The issue is that when a > scrollbar is clicked and held, if the user switches focus (ex: ALT+TAB) while > scrolling, when focused is returned to the scrolling application, the > JComboBox will

Re: RFR: 8344981: [REDO] JDK-6672644 JComboBox still scrolling if switch to another window and return back

2025-02-10 Thread Alexander Zvegintsev
On Tue, 4 Feb 2025 23:35:32 GMT, Damon Nguyen wrote: > Redo for JComboBox infinite scrolling issue. The issue is that when a > scrollbar is clicked and held, if the user switches focus (ex: ALT+TAB) while > scrolling, when focused is returned to the scrolling application, the > JComboBox will

RFR: 8349751: AIX build failure after upgrade pipewire to 1.3.81

2025-02-10 Thread Alexander Zvegintsev
Filed as a separate issue to keep the #23426 clean. Fix is the same as in the `src/java.desktop/unix/native/libpipewire/include/spa/param/audio/raw.h` part of the [JDK-8309703](https://bugs.openjdk.org/browse/JDK-8309703) - Depends on: https://git.openjdk.org/jdk/pull/23426 Commit

Re: RFR: 8348600: Update PipeWire to 1.3.81

2025-02-07 Thread Alexander Zvegintsev
On Fri, 7 Feb 2025 06:57:01 GMT, Prasanta Sadhukhan wrote: > When you say "Testing looks good" you mean the normal jtreg client testsuite > or any other specific testing that was done? normal jtreg client testsuite + manual testing - PR Comment: https://git.openjdk.org/jdk/pull/2

Re: RFR: 8347826: Introspector shows wrong method list after 8071693 [v4]

2025-02-06 Thread Alexander Zvegintsev
On Thu, 6 Feb 2025 14:10:52 GMT, Roman Marchenko wrote: >> Fixed `com.sun.beans.introspect.MethodInfo#MethodOrder` to make >> `Introspector.addMethod()` working properly when filtering methods out. >> >> Also fixed the test, and added the approptiate test case. > > Roman Marchenko has updated t

Re: RFR: 8347826: Introspector shows wrong method list after 8071693 [v3]

2025-02-06 Thread Alexander Zvegintsev
On Tue, 4 Feb 2025 16:28:51 GMT, Roman Marchenko wrote: >> Fixed `com.sun.beans.introspect.MethodInfo#MethodOrder` to make >> `Introspector.addMethod()` working properly when filtering methods out. >> >> Also fixed the test, and added the approptiate test case. > > Roman Marchenko has updated t

Re: RFR: 8348600: Update PipeWire to 1.3.81

2025-02-03 Thread Alexander Zvegintsev
On Mon, 3 Feb 2025 18:50:41 GMT, Alexander Zvegintsev wrote: > This changeset updates the pipewire headers to the latest available version. > It contains the minimum set of required header files needed to build the JDK. > > The updated headers are a direct copy from

RFR: 8348600: Update PipeWire to 1.3.81

2025-02-03 Thread Alexander Zvegintsev
This changeset updates the pipewire headers to the latest available version. It contains the minimum set of required header files needed to build the JDK. The updated headers are a direct copy from the https://gitlab.freedesktop.org/pipewire/pipewire/-/releases/1.3.81 (except for the `\t` -> `

Re: RFR: 8347836: Disabled PopupMenu shows shortcuts on Mac [v3]

2025-02-03 Thread Alexander Zvegintsev
On Mon, 3 Feb 2025 19:02:30 GMT, Damon Nguyen wrote: >> The test instructions say that disabled PopupMenus should not have shortcuts >> shown, but on MacOS, these shortcuts still appear. When checking native >> MacOS15 behavior, disabled PopupMenus still have shortcuts shown. Since the >> test

Re: RFR: 8347836: Disabled PopupMenu shows shortcuts on Mac [v2]

2025-02-03 Thread Alexander Zvegintsev
On Mon, 3 Feb 2025 17:49:15 GMT, Damon Nguyen wrote: >> The test instructions say that disabled PopupMenus should not have shortcuts >> shown, but on MacOS, these shortcuts still appear. When checking native >> MacOS15 behavior, disabled PopupMenus still have shortcuts shown. Since the >> test

Re: RFR: 8345538: Robot.mouseMove doesn't clamp bounds on macOS when trying to move mouse off screen [v11]

2025-02-03 Thread Alexander Zvegintsev
On Tue, 28 Jan 2025 23:21:26 GMT, Alisen Chung wrote: >> Currently on macOS when mouseMove is given an offscreen coordinate to move >> the mouse to, mouseMove will physically clamp to the edge of the screen, but >> if you try to grab the mouse location immediately after by using >> MouseInfo.g

Integrated: 8348675: TrayIcon tests fail in Ubuntu 24.10 Wayland

2025-02-03 Thread Alexander Zvegintsev
On Tue, 28 Jan 2025 11:28:27 GMT, Alexander Zvegintsev wrote: > Several TrayIcon tests are trying to click on the system tray icon with Robot > using XTest API. > > Basically we have the same kind of failures as before, e.g. > [JDK-8280990](https://bugs.openjdk.org/bro

Integrated: 8342096: Popup menus that request focus are not shown on Linux with Wayland

2025-01-29 Thread Alexander Zvegintsev
On Fri, 13 Dec 2024 04:16:06 GMT, Alexander Zvegintsev wrote: > The previous [JDK-8319103](https://bugs.openjdk.org/browse/JDK-8319103) fix > was not complete. > > The case where a menu item with a focusable component was not a direct child > of a window was missing(failing

Re: RFR: 8342524: Use latch in AbstractButton/bug6298940.java instead of delay [v2]

2025-01-29 Thread Alexander Zvegintsev
On Wed, 29 Jan 2025 15:46:31 GMT, Alexey Ivanov wrote: >> Use a `CountDownLatch` in `javax/swing/AbstractButton/bug6298940.java` >> instead of delay. >> Use `Util.hitMnemonics` instead of custom code to handle macOS. >> >> I ran the updated test a few times with `JTREG=REPEAT_COUNT=20`, the t

Re: RFR: 8342096: Popup menus that request focus are not shown on Linux with Wayland [v6]

2025-01-29 Thread Alexander Zvegintsev
o the ungrab event was posted prematurely. > > This can be fixed by adding `waylandWindowFocusListener` to all submenus in > hierarchy. > > The manual test updated to use this case, and also added an automated test > that checks that it didn't close prematurely. Alexande

Re: RFR: 8342096: Popup menus that request focus are not shown on Linux with Wayland [v5]

2025-01-29 Thread Alexander Zvegintsev
On Wed, 29 Jan 2025 15:50:25 GMT, Alexey Ivanov wrote: >> Alexander Zvegintsev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review comments > > test/jdk/javax/swing/JPopupMenu/NestedFocusablePo

Re: RFR: 8348675: TrayIcon tests fail in Ubuntu 24.10 Wayland [v2]

2025-01-28 Thread Alexander Zvegintsev
f >> moving/resizing/stacking the windows. > > I also tested the same tests by clicking manually instead of using the robot, > and it works as expected. > So for now, skip those tests on Wayland (and do some minor cleanup). > > Testing after modifications is also green

Re: RFR: 8348675: TrayIcon tests fail in Ubuntu 24.10 Wayland [v2]

2025-01-28 Thread Alexander Zvegintsev
On Tue, 28 Jan 2025 17:44:19 GMT, Alexey Ivanov wrote: >> Alexander Zvegintsev has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - minor >> - review comments > > test/jdk/java/awt/TrayIcon/Acti

Re: RFR: 8348648: Unnecessary Hashtable usage in javax.swing.text.html.CSS.LengthUnit

2025-01-28 Thread Alexander Zvegintsev
On Mon, 20 Jan 2025 13:07:19 GMT, Andrey Turbanov wrote: > Two `Hashtable`'s in `javax.swing.text.html.CSS.LengthUnit` class are > modified only within `` block: > 1. lengthMapping > 2. w3cLengthMapping > > We can replace them with immutable maps to avoid Hashtable synchronization > overhead.

RFR: 8348675: TrayIcon tests fails in Ubuntu 24.10 wayland

2025-01-28 Thread Alexander Zvegintsev
Several TrayIcon tests are trying to click on the system tray icon with Robot using XTest API. Basically we have the same kind of failures as before, e.g. [JDK-8280990](https://bugs.openjdk.org/browse/JDK-8280990) > the reason is this is using XTEST, an X11 protocol which will not work > outsi

Re: RFR: 8348675: TrayIcon tests fails in Ubuntu 24.10 wayland

2025-01-28 Thread Alexander Zvegintsev
On Tue, 28 Jan 2025 11:28:27 GMT, Alexander Zvegintsev wrote: > Several TrayIcon tests are trying to click on the system tray icon with Robot > using XTest API. > > Basically we have the same kind of failures as before, e.g. > [JDK-8280990](https://bugs.openjdk.org/bro

Re: RFR: 8342524: Use latch in AbstractButton/bug6298940.java instead of delay

2025-01-27 Thread Alexander Zvegintsev
On Fri, 24 Jan 2025 16:47:42 GMT, Alexey Ivanov wrote: > Use a `CountDownLatch` in `javax/swing/AbstractButton/bug6298940.java` > instead of delay. > Use `Util.hitMnemonics` instead of custom code to handle macOS. > > I ran the updated test a few times with `JTREG=REPEAT_COUNT=20`, the test

Re: RFR: 8344581: [TESTBUG] java/awt/Robot/ScreenCaptureRobotTest.java failing on macOS [v2]

2025-01-27 Thread Alexander Zvegintsev
On Mon, 27 Jan 2025 06:16:29 GMT, Naveen Narayanan wrote: >> This contains test fixes for the following issue in >> ScreenCaptureRobotTest.java >> >> •Failures in Mac OS because of mouse pointer visible on top of the image >> that is screen captured by robot. >> >> Testing: >> Tested usin

Re: RFR: 8342096: Popup menus that request focus are not shown on Linux with Wayland [v5]

2025-01-24 Thread Alexander Zvegintsev
o the ungrab event was posted prematurely. > > This can be fixed by adding `waylandWindowFocusListener` to all submenus in > hierarchy. > > The manual test updated to use this case, and also added an automated test > that checks that it didn't close prematurely. Alexande

Re: RFR: 8342096: Popup menus that request focus are not shown on Linux with Wayland [v4]

2025-01-24 Thread Alexander Zvegintsev
On Fri, 24 Jan 2025 17:30:30 GMT, Alexey Ivanov wrote: >> Alexander Zvegintsev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> getLocationOnScreen on EDT > > test/jdk/javax/swing/JPopupMenu/NestedFo

Re: RFR: 8348286: [AIX] clang 17 introduces new warning Wtentative-Definitions which produces Build errors [v3]

2025-01-23 Thread Alexander Zvegintsev
On Thu, 23 Jan 2025 13:48:06 GMT, Joachim Kern wrote: >> We (SAP) try to introduce the ‘IBM Open XL C/C++ for AIX 17.1.2’ (based on >> clang 17) as a feasible compiler for jdk25, because in combination with the >> 17.1.3 runtime this would enable the support for ubsan. >> Unfortunately, the new

Re: RFR: 8344581: [TESTBUG] java/awt/Robot/ScreenCaptureRobotTest.java failing on macOS

2025-01-23 Thread Alexander Zvegintsev
On Thu, 23 Jan 2025 11:37:23 GMT, Naveen Narayanan wrote: > This contains test fixes for the following issue in > ScreenCaptureRobotTest.java > > • Failures in Mac OS because of mouse pointer visible on top of the image > that is screen captured by robot. > > Testing: > Tested using Mach5

Integrated: 8347008: beancontext package spec does not clearly explain why the API is deprecated

2025-01-23 Thread Alexander Zvegintsev
On Tue, 21 Jan 2025 14:28:26 GMT, Alexander Zvegintsev wrote: > The beancontext package description or the deprecation text in the classes > doesn't say anything about why the API is deprecated. > A few more words have been added in this regard. This pull request has now b

Re: RFR: 8340784 : Remove PassFailJFrame constructor with screenshots

2025-01-23 Thread Alexander Zvegintsev
On Tue, 24 Dec 2024 03:02:51 GMT, anass baya wrote: > Remove PassFailJFrame constructor with screenshots since no test is using it Marked as reviewed by azvegint (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/22873#pullrequestreview-2569960923

Re: RFR: 8348299: Update List/ItemEventTest/ItemEventTest.java [v2]

2025-01-23 Thread Alexander Zvegintsev
On Thu, 23 Jan 2025 11:09:23 GMT, Alexey Ivanov wrote: >> With this changeset, I update the >> `java/awt/List/ItemEventTest/ItemEventTest.java` test to improve its >> stability and analysis of failures of the test. >> >> 1. Use thread-safe `StringBuffer` for `actualSelectionOrder` to track >>

Re: RFR: 8348299: Update List/ItemEventTest/ItemEventTest.java

2025-01-23 Thread Alexander Zvegintsev
On Wed, 22 Jan 2025 16:45:17 GMT, Alexey Ivanov wrote: > With this changeset, I update the > `java/awt/List/ItemEventTest/ItemEventTest.java` test to improve its > stability and analysis of failures of the test. > > 1. Use thread-safe `StringBuffer` for `actualSelectionOrder` to track > selec

Re: RFR: 8348308: Make fields of ListSelectionEvent final

2025-01-22 Thread Alexander Zvegintsev
On Wed, 22 Jan 2025 20:20:28 GMT, Alexey Ivanov wrote: > The fields of `ListSelectionEvent` are set in the constructor and are never > modified, mark the fields — `firstIndex`, `lastIndex`, `isAdjusting` — with > the `final` modifier. Marked as reviewed by azvegint (Reviewer). -

Re: RFR: 8348303: Remove repeated 'a' from ListSelectionEvent

2025-01-22 Thread Alexander Zvegintsev
On Wed, 22 Jan 2025 17:16:11 GMT, Alexey Ivanov wrote: > A trivial change which removes repeated ‘a’ from the [javadoc for the > `ListSelectionEvent` > class](https://docs.oracle.com/en/java/javase/21/docs/api/java.desktop/javax/swing/event/ListSelectionEvent.html): > >> “The change is limited

Re: RFR: 8342096: Popup menus that request focus are not shown on Linux with Wayland [v3]

2025-01-21 Thread Alexander Zvegintsev
On Wed, 22 Jan 2025 01:38:56 GMT, Harshitha Onkar wrote: >> Alexander Zvegintsev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> formatting > > test/jdk/javax/swing/JPopupMenu/NestedFocusablePopupTest.jav

Re: RFR: 8342096: Popup menus that request focus are not shown on Linux with Wayland [v4]

2025-01-21 Thread Alexander Zvegintsev
o the ungrab event was posted prematurely. > > This can be fixed by adding `waylandWindowFocusListener` to all submenus in > hierarchy. > > The manual test updated to use this case, and also added an automated test > that checks that it didn't close prematurely. Alexande

Re: RFR: 8347008: beancontext package spec does not clearly explain why the API is deprecated [v2]

2025-01-21 Thread Alexander Zvegintsev
> The beancontext package description or the deprecation text in the classes > doesn't say anything about why the API is deprecated. > A few more words have been added in this regard. Alexander Zvegintsev has updated the pull request incrementally with one additional commit

Re: RFR: 8347008: beancontext package spec does not clearly explain why the API is deprecated [v2]

2025-01-21 Thread Alexander Zvegintsev
On Tue, 21 Jan 2025 21:28:35 GMT, Alexey Ivanov wrote: >> Alexander Zvegintsev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review comments > > src/java.desktop/share/classes/java/beans/beancontex

RFR: 8347008: beancontext package spec does not clearly explain why the API is deprecated

2025-01-21 Thread Alexander Zvegintsev
The beancontext package description or the deprecation text in the classes doesn't say anything about why the API is deprecated. A few more words have been added in this regard. - Commit messages: - 8347008: beancontext package spec does not clearly explain why the API is deprecate

Re: RFR: 8347475: GTK: javax/swing/JColorChooser/Test8152419.java there is no swatches or RGB tab in JColorChooser [v3]

2025-01-21 Thread Alexander Zvegintsev
On Tue, 21 Jan 2025 04:09:17 GMT, Abhishek Kumar wrote: >> There are no tabs available for GTK Look and Feel due to the different >> ColorChooser UI design. Updated the test to skip testing for GTK L&F. > > Abhishek Kumar has updated the pull request incrementally with one additional > commit s

Re: RFR: 8348170: Unnecessary Hashtable usage in CSS.styleConstantToCssMap

2025-01-21 Thread Alexander Zvegintsev
On Thu, 16 Jan 2025 11:53:18 GMT, Andrey Turbanov wrote: > There is a field `javax.swing.text.html.CSS#styleConstantToCssMap` which uses > legacy `Hashtable` class. > As this map is read-only and all its content is initialized in `` we > can safely use immutable `Map` instead. Marked as review

Re: RFR: 8342096: Popup menus that request focus are not shown on Linux with Wayland [v3]

2025-01-21 Thread Alexander Zvegintsev
On Mon, 20 Jan 2025 19:35:18 GMT, Alexey Ivanov wrote: > It's an inconvenience which can be overcome by compiling the test and > `SkippedException` together: *Grumbling*: "Overcome" is a good word in this case, as it's not even close to the convenience of just `java `, where you don't have to

Re: RFR: 8342096: Popup menus that request focus are not shown on Linux with Wayland [v3]

2025-01-21 Thread Alexander Zvegintsev
On Mon, 20 Jan 2025 19:56:18 GMT, Alexey Ivanov wrote: > What if the window that loses focus is not the parent of the popup hierarchy? > Or is it guaranteed that all the children of the popup window receive the > focus lost event? I can't imagine such a case, but even if it is, it will all be

Re: RFR: 8342096: Popup menus that request focus are not shown on Linux with Wayland [v3]

2025-01-21 Thread Alexander Zvegintsev
o the ungrab event was posted prematurely. > > This can be fixed by adding `waylandWindowFocusListener` to all submenus in > hierarchy. > > The manual test updated to use this case, and also added an automated test > that checks that it didn't close prematurely. Alexande

Re: RFR: 8342096: Popup menus that request focus are not shown on Linux with Wayland [v2]

2025-01-19 Thread Alexander Zvegintsev
On Mon, 20 Jan 2025 01:08:01 GMT, Alexander Zvegintsev wrote: >> The previous [JDK-8319103](https://bugs.openjdk.org/browse/JDK-8319103) fix >> was not complete. >> >> The case where a menu item with a focusable component was not a direct child >> of a wind

Re: RFR: 8342096: Popup menus that request focus are not shown on Linux with Wayland [v2]

2025-01-19 Thread Alexander Zvegintsev
o the ungrab event was posted prematurely. > > This can be fixed by adding `waylandWindowFocusListener` to all submenus in > hierarchy. > > The manual test updated to use this case, and also added an automated test > that checks that it didn't close prematurely. Alexande

Re: RFR: 8342096: Popup menus that request focus are not shown on Linux with Wayland [v2]

2025-01-19 Thread Alexander Zvegintsev
On Fri, 13 Dec 2024 22:12:58 GMT, Alexander Zvegintsev wrote: >> src/java.desktop/unix/classes/sun/awt/UNIXToolkit.java line 550: >> >>> 548: >>> addWaylandWindowFocusListenerToWindow(oppositeWindow); >>> 549:

Re: RFR: 8345618: javax/swing/text/Caret/8163124/CaretFloatingPointAPITest.java leaves Caret is not complete

2025-01-17 Thread Alexander Zvegintsev
On Wed, 15 Jan 2025 09:45:33 GMT, Prasanta Sadhukhan wrote: > This manual test draws a "custom" Caret which is to be placed in the same > position between characters but it is seen the movement of caret leaves > artifacts. Custom caret is rendering 2 horizontal and 1 vertical line to > render

Re: RFR: 8282862: AwtWindow::SetIconData leaks old icon handles if an exception is detected [v4]

2025-01-17 Thread Alexander Zvegintsev
On Wed, 15 Jan 2025 21:52:52 GMT, Rajat Mahajan wrote: >> **Issue:** >> AwtWindow::SetIconData leaks the old icon handles in hOldIcon and hOldIconSm >> if CreateIconFromRaster raises an exception. Additionally, an exception is >> checked only after the first call to CreateIconFromRaster. >> >

Re: RFR: 8347911: Limit the length of inflated text chunks

2025-01-16 Thread Alexander Zvegintsev
On Thu, 16 Jan 2025 11:52:14 GMT, Jayathirth D V wrote: > In PNG we have non-essential compressed text chunks zTXt and iTXt, which can > consume considerable amount of memory when they contain large amount of > compressed data. > > We should restrict the amount of text data we want to decompre

Re: RFR: 8347916: Simplify javax.swing.text.html.CSS.LengthUnit.getValue

2025-01-16 Thread Alexander Zvegintsev
On Fri, 1 Nov 2024 14:43:39 GMT, Andrey Turbanov wrote: > Static and non-static `CSS.LengthUnit.getValue` methods essentially do the > same thing. > https://github.com/openjdk/jdk/blob/751a914b0a377d4e1dd30d2501f0ab4e327dea34/src/java.desktop/share/classes/javax/swing/text/html/CSS.java#L3085-L3

Re: RFR: 8347370: Unnecessary Hashtable usage in javax.swing.text.html.HTML

2025-01-14 Thread Alexander Zvegintsev
On Fri, 1 Nov 2024 20:13:31 GMT, Andrey Turbanov wrote: > The Hashtable `javax.swing.text.html.HTML.scMapping` is modified only within > `` block. > We can replace it with immutable map to avoid Hashtable synchronization > overhead. Marked as reviewed by azvegint (Reviewer). - PR

Re: RFR: 8336920: ArithmeticException in javax.sound.sampled.AudioInputStream

2025-01-13 Thread Alexander Zvegintsev
On Mon, 13 Jan 2025 10:23:28 GMT, Alexander Zuev wrote: > If pitch drifts too low assume minimum allowed pitch to continue sampling. Marked as reviewed by azvegint (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/23064#pullrequestreview-254607

Re: RFR: 8346036: Unnecessary Hashtable usage in javax.swing.text.html.parser.Entity

2025-01-08 Thread Alexander Zvegintsev
On Fri, 1 Nov 2024 14:02:52 GMT, Andrey Turbanov wrote: > The Hashtable `javax.swing.text.html.parser.Entity#entityTypes` is modified > only within `` block. > We can replace it with immutable map to avoid Hashtable synchronization > overhead. > Similar to what we did in #21825. Marked as revi

Re: RFR: 8346953: Remove unnecessary @SuppressWarnings annotations (client, #2)

2025-01-06 Thread Alexander Zvegintsev
On Thu, 2 Jan 2025 20:01:28 GMT, Archie Cobbs wrote: > Please review this patch which removes unnecessary `@SuppressWarnings` > annotations. > > This issue is a follow up to > [JDK-8343476](https://bugs.openjdk.org/browse/JDK-8343476) to remove some > additional warnings that were missed in t

Re: RFR: 8334644 : Automate javax/print/attribute/PageRangesException.java [v2]

2025-01-06 Thread Alexander Zvegintsev
On Mon, 6 Jan 2025 05:43:19 GMT, Renjith Kannath Pariyangad wrote: >> Hi All, >> >> I have updated the test and made it automation friendly. Please review and >> let me know your suggestions > > Renjith Kannath Pariyangad has updated the pull request incrementally with > one additional commit

Re: RFR: 8342096: Popup menus that request focus are not shown on Linux with Wayland

2024-12-13 Thread Alexander Zvegintsev
On Fri, 13 Dec 2024 12:39:02 GMT, Alexey Ivanov wrote: >> The previous [JDK-8319103](https://bugs.openjdk.org/browse/JDK-8319103) fix >> was not complete. >> >> The case where a menu item with a focusable component was not a direct child >> of a window was missing(failing the `if (window == >

Re: RFR: 8343224: print/Dialog/PaperSizeError.java fails with MediaSizeName is not A4: A4 [v5]

2024-12-12 Thread Alexander Zvegintsev
On Thu, 12 Dec 2024 13:29:55 GMT, GennadiyKrivoshein wrote: >> Fix for https://bugs.openjdk.org/browse/JDK-8343224, >> print/Dialog/PaperSizeError.java fails with MediaSizeName is not A4: A4. >> >> **Reason of the bug:** >> The reason for the regression bug is a missed check for the standard me

RFR: 8342096: Popup menus that request focus are not shown on Linux with Wayland

2024-12-12 Thread Alexander Zvegintsev
The previous [JDK-8319103](https://bugs.openjdk.org/browse/JDK-8319103) fix was not complete. The case where a menu item with a focusable component was not a direct child of a window was missing(failing the `if (window == oppositeWindow.getParent() ) {` check), so the ungrab event was posted pr

Re: RFR: 8343224: print/Dialog/PaperSizeError.java fails with MediaSizeName is not A4: A4 [v4]

2024-12-12 Thread Alexander Zvegintsev
On Wed, 4 Dec 2024 20:38:52 GMT, GennadiyKrivoshein wrote: >> Fix for https://bugs.openjdk.org/browse/JDK-8343224, >> print/Dialog/PaperSizeError.java fails with MediaSizeName is not A4: A4. >> >> **Reason of the bug:** >> The reason for the regression bug is a missed check for the standard med

Re: RFR: 8345616: Unnecessary Hashtable usage in javax.swing.text.html.parser.Element

2024-12-11 Thread Alexander Zvegintsev
On Fri, 1 Nov 2024 13:32:07 GMT, Andrey Turbanov wrote: > The Hashtable `javax.swing.text.html.parser.Element#contentTypes` is > initialized only within `` block. > We can replace it with immutable map to avoid Hashtable `synchronized` > overhead. Marked as reviewed by azvegint (Reviewer). --

Re: RFR: 8345876: Update nativeAddAtIndex comment to match the code [v2]

2024-12-11 Thread Alexander Zvegintsev
On Wed, 11 Dec 2024 09:13:06 GMT, Saagar Jha wrote: >> 8345876: Update nativeAddAtIndex comment to match the code > > Saagar Jha has updated the pull request with a new target base due to a merge > or a rebase. The pull request now contains one commit: > > Update nativeAddAtIndex comment to m

Re: RFR: 8345888: Broken links in the JDK 24 JavaDoc API documentation, build 27

2024-12-10 Thread Alexander Zvegintsev
On Tue, 10 Dec 2024 11:43:44 GMT, Nizar Benalla wrote: > Please review this trivial change fixing a broken link in > `PresentationDirection.java` > > The contents of the links are identical, I used the [wayback > machine](https://web.archive.org/web/20210511075617/https://ftp.pwg.org/pub/pwg/s

Re: RFR: 8345876: Update nativeAddAtIndex comment to match the code

2024-12-10 Thread Alexander Zvegintsev
On Sat, 7 Dec 2024 04:41:01 GMT, Saagar Jha wrote: > 8345876: Update nativeAddAtIndex comment to match the code > Ok so I think that solved the "I don't have a way to make a bug for this" > issue. I have signed an OCA a while back, is there anything else I need to do > there? It looks like y

Re: RFR: 8345144: Robot does not specify all causes of IllegalThreadStateException

2024-12-10 Thread Alexander Zvegintsev
On Mon, 9 Dec 2024 20:47:07 GMT, Harshitha Onkar wrote: > ``` > public void delay(int ms) { > if (isEventDispatchThread()) { > throw new IllegalThreadStateException("delay() called on EDT"); > } > ``` There is already a `checkNotDispatchThread` method, so we can reuse it. --

Re: RFR: 8343170: java/awt/Cursor/JPanelCursorTest/JPanelCursorTest.java does not show the default cursor [v4]

2024-12-10 Thread Alexander Zvegintsev
On Wed, 20 Nov 2024 18:10:40 GMT, Damon Nguyen wrote: >> Change test instructions and drawString to say "Text" instead of "Default" >> for cursor type. Added instructions to test `setCursor` for the frame as >> well. Also increased the column count to fit the instructions in linux, as >> it wa

Re: RFR: 8344668: Unnecessary array allocations and copying in TextLine [v3]

2024-12-06 Thread Alexander Zvegintsev
On Thu, 5 Dec 2024 20:27:16 GMT, Daniel Gredler wrote: >> When `LineBreakMeasurer` is used to break text into lines, internally it >> uses `TextMeasurer` and `TextLine` to do the job. In the common case, >> `TextLine.getComponents(...)` allocates a `TextLineComponent[]` with room >> for a sing

Re: RFR: 8334581: Remove no-arg constructor BasicSliderUI()

2024-12-06 Thread Alexander Zvegintsev
On Fri, 6 Dec 2024 06:37:28 GMT, Prasanta Sadhukhan wrote: > The no-arg constructor BasicSliderUI() was added accidentally. It should be > removed. > > [JDK-8334580](https://bugs.openjdk.org/browse/JDK-8334580) deprecates the > BasicSliderUI() constructor, and this follow-up is to remove the

Re: RFR: 8340580: Characters in Document diagrams not in the middle

2024-12-04 Thread Alexander Zvegintsev
On Wed, 4 Dec 2024 15:40:31 GMT, Alexey Ivanov wrote: > This is a continuation to #20376, it moves the text in > `Document-{insert,remove}.svg` down by 0.5 pixel. > > This change makes the text centred in the boxes. > > Here are quotes from > https://github.com/openjdk/jdk/pull/20376#issuecom

Re: RFR: 8324491: Keyboard layout did not keep it state if it was changed when dialog is active

2024-12-03 Thread Alexander Zvegintsev
On Wed, 27 Nov 2024 12:47:29 GMT, Dmitry Markov wrote: > If AWT/Swing app displays several windows and an user changes the keyboard > layout and then closes the focused window, the app does not keep the keyboard > layout changes. > > It is necessary to sync currentLocale and Windows keyboard l

Re: RFR: JDK-8344368 : IndependenceSwingTest.java and IndependenceAWTTest.java failed: Selected text & clipboard contents differs [v5]

2024-12-03 Thread Alexander Zvegintsev
On Tue, 3 Dec 2024 17:17:22 GMT, Harshitha Onkar wrote: >> The following two tests failed intermittently due to stability issues after >> JEP486 (Removal of Security Manager). Tests have been updated for stability. >> This test is applicable for linux platform only, hence `@requires (os.family

Re: RFR: JDK-8344368 : IndependenceSwingTest.java and IndependenceAWTTest.java failed: Selected text & clipboard contents differs [v3]

2024-12-03 Thread Alexander Zvegintsev
On Tue, 3 Dec 2024 01:13:09 GMT, Alexander Zvegintsev wrote: >> Harshitha Onkar has updated the pull request incrementally with one >> additional commit since the last revision: >> >> frame title > > test/jdk/java/awt/datatransfer/Independence/Independence

Re: RFR: JDK-8344368 : IndependenceSwingTest.java and IndependenceAWTTest.java failed: Selected text & clipboard contents differs [v4]

2024-12-03 Thread Alexander Zvegintsev
On Tue, 3 Dec 2024 05:25:18 GMT, Harshitha Onkar wrote: >> The following two tests failed intermittently due to stability issues after >> JEP486 (Removal of Security Manager). Tests have been updated for stability. >> This test is applicable for linux platform only, hence `@requires (os.family

Re: RFR: 8345141: Remove uses of SecurityManager in ShellFolder related classes [v3]

2024-12-02 Thread Alexander Zvegintsev
On Mon, 2 Dec 2024 23:03:14 GMT, Phil Race wrote: >> Remove SecurityManager related code from Swing filechooser implementation > > Phil Race has updated the pull request incrementally with one additional > commit since the last revision: > > 8345141 Marked as reviewed by azvegint (Reviewer).

Re: RFR: JDK-8344368 : IndependenceSwingTest.java and IndependenceAWTTest.java failed: Selected text & clipboard contents differs [v3]

2024-12-02 Thread Alexander Zvegintsev
On Tue, 3 Dec 2024 00:17:11 GMT, Harshitha Onkar wrote: >> The following two tests failed intermittently due to stability issues after >> JEP486 (Removal of Security Manager). Tests have been updated for stability. >> This test is applicable for linux platform only, hence `@requires (os.family

Re: RFR: 8345143: Remove uses of SecurityManager in the java.desktop module [v6]

2024-12-02 Thread Alexander Zvegintsev
On Mon, 2 Dec 2024 19:01:06 GMT, Phil Race wrote: >> Remove SecurityManager related code in the desktop module that is not >> covered by other PRs > > Phil Race has updated the pull request incrementally with one additional > commit since the last revision: > > 8345143 Marked as reviewed by

  1   2   3   4   5   6   7   8   >