Re: RFR: 8338282: javax/swing/JMenuBar/TestMenuMnemonicLinuxAndMac.java test failed on macOS and Ubuntu [v2]

2025-07-15 Thread Abhishek Kumar
On Wed, 16 Jul 2025 06:20:22 GMT, Tejesh R wrote: >> `frame.requestFocus()` tries to make the frame current focus owner but it >> does not guarantee that focus will be granted. So, to be on safer side, it's >> better to bring the frame to the front and the request keyboard focus. > > According

Re: RFR: 8338282: javax/swing/JMenuBar/TestMenuMnemonicLinuxAndMac.java test failed on macOS and Ubuntu [v2]

2025-07-15 Thread Abhishek Kumar
On Wed, 16 Jul 2025 05:48:43 GMT, SendaoYan wrote: >> Abhishek Kumar has updated the pull request incrementally with one >> additional commit since the last revision: >> >> copyright year update > > test/jdk/javax/swing/JMenuBar/TestMenuMnemonicLinuxAndMac.java line 120: > >> 118: fr

Re: RFR: 8338282: javax/swing/JMenuBar/TestMenuMnemonicLinuxAndMac.java test failed on macOS and Ubuntu [v2]

2025-07-15 Thread Tejesh R
On Wed, 16 Jul 2025 06:12:11 GMT, Abhishek Kumar wrote: >> test/jdk/javax/swing/JMenuBar/TestMenuMnemonicLinuxAndMac.java line 120: >> >>> 118: frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); >>> 119: frame.setVisible(true); >>> 120: frame.toFront(); >> >> ` frame.

Re: RFR: 8338282: javax/swing/JMenuBar/TestMenuMnemonicLinuxAndMac.java test failed on macOS and Ubuntu [v2]

2025-07-15 Thread Abhishek Kumar
> Test is failing on intermittently on macOS and ubuntu machine in CI testing. > Test stabilization done by requesting focus for the frame before starting the > testing. > > Ran the test before and after the fix and there is no failure after test > stabilization for multiple run. > CI link is a

Re: RFR: 8338282: javax/swing/JMenuBar/TestMenuMnemonicLinuxAndMac.java test failed on macOS and Ubuntu

2025-07-15 Thread Abhishek Kumar
On Wed, 16 Jul 2025 05:25:39 GMT, Tejesh R wrote: >> Test is failing on intermittently on macOS and ubuntu machine in CI testing. >> Test stabilization done by requesting focus for the frame before starting >> the testing. >> >> Ran the test before and after the fix and there is no failure aft

Re: RFR: 8338282: javax/swing/JMenuBar/TestMenuMnemonicLinuxAndMac.java test failed on macOS and Ubuntu

2025-07-15 Thread SendaoYan
On Tue, 15 Jul 2025 05:40:17 GMT, Abhishek Kumar wrote: > Test is failing on intermittently on macOS and ubuntu machine in CI testing. > Test stabilization done by requesting focus for the frame before starting the > testing. > > Ran the test before and after the fix and there is no failure af

Re: RFR: 8338282: javax/swing/JMenuBar/TestMenuMnemonicLinuxAndMac.java test failed on macOS and Ubuntu

2025-07-15 Thread Tejesh R
On Tue, 15 Jul 2025 05:40:17 GMT, Abhishek Kumar wrote: > Test is failing on intermittently on macOS and ubuntu machine in CI testing. > Test stabilization done by requesting focus for the frame before starting the > testing. > > Ran the test before and after the fix and there is no failure af

Re: RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v33]

2025-07-15 Thread Prasanta Sadhukhan
On Tue, 15 Jul 2025 19:31:54 GMT, Phil Race wrote: > No issue was raised because no one knew any better. Now we know. But people would have noted the difference if native app renders it differently in Windows 10..Now both bullet/checkmark and imageicon is superimposed on top of each other..if

[jdk25] Integrated: Merge 121f5a72e4c23919b3a3b474cc3f1ac29ec611af

2025-07-15 Thread Brent Christian
On Wed, 16 Jul 2025 03:40:43 GMT, Brent Christian wrote: > This brings in cpu25_07 changes. This pull request has now been integrated. Changeset: 0e6bf005 Author:Brent Christian URL: https://git.openjdk.org/jdk/commit/0e6bf0055057fae844748a300551549553f59f03 Stats: 358 lines in

Re: [jdk25] RFR: Merge 121f5a72e4c23919b3a3b474cc3f1ac29ec611af [v2]

2025-07-15 Thread Brent Christian
> This brings in cpu25_07 changes. Brent Christian has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. - Changes: - all: https://git.openjdk.org/jdk/pull/26340/files

Re: [jdk25] RFR: Merge 121f5a72e4c23919b3a3b474cc3f1ac29ec611af

2025-07-15 Thread Jaikiran Pai
On Wed, 16 Jul 2025 03:40:43 GMT, Brent Christian wrote: > This brings in cpu25_07 changes. Marked as reviewed by jpai (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/26340#pullrequestreview-3023001849

[jdk25] RFR: Merge 121f5a72e4c23919b3a3b474cc3f1ac29ec611af

2025-07-15 Thread Brent Christian
This brings in cpu25_07 changes. - Commit messages: - 8360147: Better Glyph drawing redux - 8355884: [macos] java/awt/Frame/I18NTitle.java fails on MacOS - 8350991: Improve HTTP client header handling - 8349594: Enhance TLS protocol support - 8349584: Improve compiler processing

Re: RFR: 8362289: [macOS] Remove finalize method in JRSUIControls.java

2025-07-15 Thread Sergey Bylokhov
On Tue, 15 Jul 2025 22:34:21 GMT, Phil Race wrote: >> there is even a bug for Cleaner that the reachabilityFence should be moved >> to the end of register: >> https://bugs.openjdk.org/browse/JDK-8291867 >> If there are no objectiosn I'll add it to the Disposer.addRecord in a >> separate bug. >

Re: RFR: 8362289: [macOS] Remove finalize method in JRSUIControls.java

2025-07-15 Thread Sergey Bylokhov
On Tue, 15 Jul 2025 22:38:07 GMT, Phil Race wrote: >> src/java.desktop/macosx/classes/apple/laf/JRSUIControl.java line 138: >> >>> 136: } >>> 137: cfDictionaryPtr = 0; >>> 138: } >> >> Is it possible for this method to be called twice? It seems that not all >> o

Re: RFR: 8362291: [macOS] Remove finalize method in CGraphicsEnvironment.java

2025-07-15 Thread Sergey Bylokhov
On Tue, 15 Jul 2025 22:39:48 GMT, Phil Race wrote: >> src/java.desktop/macosx/classes/sun/awt/CGraphicsEnvironment.java line 153: >> >>> 151: } >>> 152: >>> 153: public synchronized void dispose() { >> >> Why the method is synchronized? we can call it on a different threads via

Re: RFR: 8362291: [macOS] Remove finalize method in CGraphicsEnvironment.java

2025-07-15 Thread Phil Race
On Tue, 15 Jul 2025 21:14:54 GMT, Sergey Bylokhov wrote: >> Remove a finalize() method in CGraphicsEnvironment, replacing it with >> Disposer. >> >> I don't see a way to add a test to verify this clean up. > > src/java.desktop/macosx/classes/sun/awt/CGraphicsEnvironment.java line 153: > >> 151

Re: RFR: 8362289: [macOS] Remove finalize method in JRSUIControls.java

2025-07-15 Thread Phil Race
On Tue, 15 Jul 2025 20:24:46 GMT, Sergey Bylokhov wrote: >> Eliminate a finalize() method in the Swing macOS implementation. >> >> I tested that the dispose method is being called by running this small test >> in combination with some 'println' statements in the source code (now >> removed) >>

Re: RFR: 8362289: [macOS] Remove finalize method in JRSUIControls.java

2025-07-15 Thread Phil Race
On Tue, 15 Jul 2025 21:43:04 GMT, Sergey Bylokhov wrote: >> src/java.desktop/macosx/classes/apple/laf/JRSUIControl.java line 115: >> >>> 113: flipped = other.flipped; >>> 114: cfDictionaryPtr = getCFDictionary(flipped); >>> 115: if (cfDictionaryPtr == 0) throw new Runtime

Re: RFR: 8362289: [macOS] Remove finalize method in JRSUIControls.java

2025-07-15 Thread Sergey Bylokhov
On Tue, 15 Jul 2025 20:24:36 GMT, Sergey Bylokhov wrote: >> Eliminate a finalize() method in the Swing macOS implementation. >> >> I tested that the dispose method is being called by running this small test >> in combination with some 'println' statements in the source code (now >> removed) >>

Re: RFR: 8338282: javax/swing/JMenuBar/TestMenuMnemonicLinuxAndMac.java test failed on macOS and Ubuntu

2025-07-15 Thread Sergey Bylokhov
On Tue, 15 Jul 2025 05:40:17 GMT, Abhishek Kumar wrote: > Test is failing on intermittently on macOS and ubuntu machine in CI testing. > Test stabilization done by requesting focus for the frame before starting the > testing. > > Ran the test before and after the fix and there is no failure af

Re: RFR: 8362291: [macOS] Remove finalize method in CGraphicsEnvironment.java

2025-07-15 Thread Sergey Bylokhov
On Tue, 15 Jul 2025 20:42:15 GMT, Phil Race wrote: > Remove a finalize() method in CGraphicsEnvironment, replacing it with > Disposer. > > I don't see a way to add a test to verify this clean up. src/java.desktop/macosx/classes/sun/awt/CGraphicsEnvironment.java line 153: > 151: } > 15

RFR: 8362291: [macOS] Remove finalize method in CGraphicsEnvironment.java

2025-07-15 Thread Phil Race
Remove a finalize() method in CGraphicsEnvironment, replacing it with Disposer. I don't see a way to add a test to verify this clean up. - Commit messages: - 8362291 Changes: https://git.openjdk.org/jdk/pull/26332/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=26332&range

Integrated: 8358468: Enhance code consistency: java.desktop/macos

2025-07-15 Thread Sergey Bylokhov
On Tue, 3 Jun 2025 06:38:25 GMT, Sergey Bylokhov wrote: > This PR updates the last (!!) remaining platform-specific part of the > java.desktop module: macos. > The unix/windows were previously addressed in: > https://github.com/openjdk/jdk/pull/24170 , > https://github.com/openjdk/jdk/pull/249

Re: RFR: 8362289: [macOS] Remove finalize method in JRSUIControls.java

2025-07-15 Thread Sergey Bylokhov
On Tue, 15 Jul 2025 19:57:56 GMT, Phil Race wrote: > Eliminate a finalize() method in the Swing macOS implementation. > > I tested that the dispose method is being called by running this small test > in combination with some 'println' statements in the source code (now removed) > > import java

Re: RFR: 8362289: [macOS] Remove finalize method in JRSUIControls.java

2025-07-15 Thread Phil Race
On Tue, 15 Jul 2025 19:57:56 GMT, Phil Race wrote: > Eliminate a finalize() method in the Swing macOS implementation. > > I tested that the dispose method is being called by running this small test > in combination with some 'println' statements in the source code (now removed) > > import java

RFR: 8362289: [macOS] Remove finalize method in JRSUIControls.java

2025-07-15 Thread Phil Race
Eliminate a finalize() method in the Swing macOS implementation. I tested that the dispose method is being called by running this small test in combination with some 'println' statements in the source code (now removed) import javax.swing.JTabbedPane; import javax.swing.plaf.TabbedPaneUI; publi

Re: RFR: 8361521 : BogusFocusableWindowState.java fails with StackOverflowError on Linux

2025-07-15 Thread Alisen Chung
On Mon, 14 Jul 2025 17:00:35 GMT, Anass Baya wrote: > The getFocusableWindowState() method is not intended to modify the > configuration; doing so can cause recursive re-entry on Linux. What's causing the recursive re-entry? Is the bug in the code path in linux when calling getFocusableWindowS

Re: RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v33]

2025-07-15 Thread Phil Race
On Mon, 14 Jul 2025 05:16:11 GMT, Prasanta Sadhukhan wrote: > > Shouldn't Windows 10 do it the new way too ? > > I didn't intentionally change the design for Windows 10 as that was the > behaviour which was in existence (even though by "accident" as we learnt) for > so long our JDK code for W

Re: RFR: 8360070: AccessibleText.getBeforeIndex returns null for last character [v3]

2025-07-15 Thread Nikita Provotorov
On Tue, 15 Jul 2025 03:58:55 GMT, Sergey Bylokhov wrote: >> I modified Abhishek's test: >> >> >> System.out.println(" i bef at aft"); >> for (int i : new int[] {0, 1, 2, 8, 9, 10}) { >> System.out.printf("%2d%5s%5s%5s\n", >>

Re: RFR: 8361283: [Accessibility, macOS, VoiceOver] VoiceOver announced Tab items of JTabbedPane as RadioButton on macOS [v4]

2025-07-15 Thread Artem Semenov
On Mon, 14 Jul 2025 08:36:57 GMT, Abhishek Kumar wrote: >> **Issue** : VO doesn't announce Tab items of a JTabbedPane as RadioButton >> which is wrong. >> >> **Analysis** : That is due to the role mapping of tab items as >> `NSAccessibilityRadioButtonRole` in `JavaAccessibilityUtilities.m` fi

Re: RFR: 8360120: Bundled macOS applications not receiving OpenURL events when launched as subprocess [v2]

2025-07-15 Thread Dmitry Kulikov
On Wed, 25 Jun 2025 07:03:33 GMT, Dmitry Kulikov wrote: >> Added an on-demand installation of the native OpenURL event handler to the >> `Application.setOpenURIHandler()`. >> >> This does not break the specification for the affected API, since the >> requirement of the application being bundle