[jfx20u] Integrated: 8282359: Intermittent WebKit build failure on Windows: C1090: PDB API call failed, error code 23

2023-04-10 Thread Hima Bindu Meda
On Mon, 10 Apr 2023 17:21:34 GMT, Hima Bindu Meda wrote: > Clean Backport.This pull request contains a backport of commit > [a264435d](https://github.com/openjdk/jfx/commit/a264435dccba6ec386548f76f1ace095d943f4ca) > from the [openjdk/jfx](https://git.openjdk.org/jfx) repository. This pull req

Update javadoc tool version?

2023-04-10 Thread Nir Lisker
Hi, I noticed that the JDK's javadoc contains a 'NEW' section [1], but JavaFX's docs don't have it. Do we need to update the javadoc tool? [1] https://docs.oracle.com/en/java/javase/19/docs/api/new-list.html - Nir

Re: RFR: JDK-8303897 ObservableValue's when binding should only invalidate when strictly needed [v3]

2023-04-10 Thread Nir Lisker
On Mon, 10 Apr 2023 00:14:31 GMT, John Hendrikx wrote: > Well, I am just following the specification here, but JavaFX itself isn't... > > We may need to make a decision here. JavaFX is not consistent itself where it > applies `equals` and where it uses reference equality: Alright, so this is f

Re: RFR: JDK-8224260: ChangeListener not triggered when adding a new listener in invalidated method

2023-04-10 Thread Nir Lisker
On Thu, 30 Mar 2023 21:53:48 GMT, John Hendrikx wrote: > Fixes three issues in ExpressionHelper: > > - Current Value was not retained when changing from SingleChange to Generic, > this can lead to missed changes > - Current Value was not retained when changing from Generic to SingleChange, > t

[jfx20u] Integrated: 8286089: Intermittent WebKit build failure on macOS in JavaScriptCore

2023-04-10 Thread Jay Bhaskar
On Mon, 10 Apr 2023 17:19:15 GMT, Jay Bhaskar wrote: > clean backport to jfx20u. The fix solves the build issue on the macOx11_64 > machine. It fixes the intermittent build issue. The build issue occurs > without the fix and does not occur with the fix. This pull request has now been integrate

Re: RFR: 8278938: [Win] Robot can target wrong key for punctuation and symbols [v4]

2023-04-10 Thread Kevin Rushforth
On Mon, 10 Apr 2023 23:55:51 GMT, Martin Fox wrote: >> When processing a `WM_CHAR` event on an OEM key (punctuation, symbol, dead >> key) the glass code will dynamically query the key's unshifted character to >> determine the Java code to assign to it. This is necessary since the >> relationsh

Re: RFR: 8278938: [Win] Robot can target wrong key for punctuation and symbols [v4]

2023-04-10 Thread Martin Fox
> When processing a `WM_CHAR` event on an OEM key (punctuation, symbol, dead > key) the glass code will dynamically query the key's unshifted character to > determine the Java code to assign to it. This is necessary since the > relationship between OEM key codes and the characters they generate

Integrated: JDK-8304933: BitSet (used for CSS pseudo class states) listener management is incorrect

2023-04-10 Thread John Hendrikx
On Mon, 27 Mar 2023 14:24:42 GMT, John Hendrikx wrote: > BitSet uses the SetListenerHelper abstraction to prevent allocating the > listener arrays. > > When removing listeners, the newly returned listener helper (which may be > different from the one called) is not reassigned. This effectively

Re: RFR: JDK-8304933: BitSet (used for CSS pseudo class states) listener management is incorrect [v2]

2023-04-10 Thread Kevin Rushforth
On Mon, 10 Apr 2023 23:08:31 GMT, John Hendrikx wrote: >> BitSet uses the SetListenerHelper abstraction to prevent allocating the >> listener arrays. >> >> When removing listeners, the newly returned listener helper (which may be >> different from the one called) is not reassigned. This effect

Re: RFR: JDK-8304933: BitSet (used for CSS pseudo class states) listener management is incorrect [v2]

2023-04-10 Thread Michael Strauß
On Mon, 10 Apr 2023 23:08:31 GMT, John Hendrikx wrote: >> BitSet uses the SetListenerHelper abstraction to prevent allocating the >> listener arrays. >> >> When removing listeners, the newly returned listener helper (which may be >> different from the one called) is not reassigned. This effect

Re: RFR: JDK-8304933: BitSet (used for CSS pseudo class states) listener management is incorrect [v2]

2023-04-10 Thread John Hendrikx
On Mon, 10 Apr 2023 22:07:41 GMT, John Hendrikx wrote: >> As with the earlier comment, any changes to the behavior of a null listener >> should be done separately and not part of this PR. > > Alright, I'll file another ticket for that. The general state of this class > is quite poor, as it doe

Re: RFR: JDK-8304933: BitSet (used for CSS pseudo class states) listener management is incorrect [v2]

2023-04-10 Thread John Hendrikx
> BitSet uses the SetListenerHelper abstraction to prevent allocating the > listener arrays. > > When removing listeners, the newly returned listener helper (which may be > different from the one called) is not reassigned. This effectively means that > removing the listener does not happen. >

Re: RFR: 8301312: Create implementation of NSAccessibilityButton protocol

2023-04-10 Thread Kevin Rushforth
On Thu, 6 Apr 2023 20:43:13 GMT, Alexander Zuev wrote: > Add the common base component for all the new implementing native classes > Change native peer creation to use the new base component The new code will > instantiate new protocol implementation for the given role if it exists or > an ol

Re: RFR: 8278938: [Win] Robot can target wrong key for punctuation and symbols [v2]

2023-04-10 Thread Kevin Rushforth
On Fri, 7 Apr 2023 18:28:28 GMT, Martin Fox wrote: >> PR looks good and works fine on Windows 11. >> >> I've tested the test attached to the JBS issue with a Spanish keyboard. It >> fails without the patch for `+`, `'`, `¡`, and passes with it. >> >> There is a NPE if you press AltGr (with or

Re: RFR: 8278938: [Win] Robot can target wrong key for punctuation and symbols [v3]

2023-04-10 Thread Kevin Rushforth
On Fri, 7 Apr 2023 17:19:56 GMT, Martin Fox wrote: >> When processing a `WM_CHAR` event on an OEM key (punctuation, symbol, dead >> key) the glass code will dynamically query the key's unshifted character to >> determine the Java code to assign to it. This is necessary since the >> relationshi

Re: RFR: JDK-8304933: BitSet (used for CSS pseudo class states) listener management is incorrect

2023-04-10 Thread John Hendrikx
On Mon, 10 Apr 2023 20:45:22 GMT, Kevin Rushforth wrote: >> modules/javafx.graphics/src/main/java/com/sun/javafx/css/BitSet.java line >> 617: >> >>> 615: @Override >>> 616: public void removeListener(InvalidationListener >>> invalidationListener) { >>> 617: if (invalidationList

Re: RFR: JDK-8304933: BitSet (used for CSS pseudo class states) listener management is incorrect

2023-04-10 Thread Kevin Rushforth
On Mon, 10 Apr 2023 21:14:02 GMT, Michael Strauß wrote: >> BitSet uses the SetListenerHelper abstraction to prevent allocating the >> listener arrays. >> >> When removing listeners, the newly returned listener helper (which may be >> different from the one called) is not reassigned. This effec

Re: RFR: JDK-8304933: BitSet (used for CSS pseudo class states) listener management is incorrect

2023-04-10 Thread Kevin Rushforth
On Mon, 27 Mar 2023 14:24:42 GMT, John Hendrikx wrote: > BitSet uses the SetListenerHelper abstraction to prevent allocating the > listener arrays. > > When removing listeners, the newly returned listener helper (which may be > different from the one called) is not reassigned. This effectively

Re: RFR: JDK-8304933: BitSet (used for CSS pseudo class states) listener management is incorrect

2023-04-10 Thread Michael Strauß
On Mon, 27 Mar 2023 14:24:42 GMT, John Hendrikx wrote: > BitSet uses the SetListenerHelper abstraction to prevent allocating the > listener arrays. > > When removing listeners, the newly returned listener helper (which may be > different from the one called) is not reassigned. This effectively

Re: RFR: JDK-8304933: BitSet (used for CSS pseudo class states) listener management is incorrect

2023-04-10 Thread Kevin Rushforth
On Mon, 3 Apr 2023 06:29:58 GMT, Michael Strauß wrote: >> BitSet uses the SetListenerHelper abstraction to prevent allocating the >> listener arrays. >> >> When removing listeners, the newly returned listener helper (which may be >> different from the one called) is not reassigned. This effect

Re: RFR: JDK-8304933: BitSet (used for CSS pseudo class states) listener management is incorrect

2023-04-10 Thread Kevin Rushforth
On Mon, 27 Mar 2023 14:24:42 GMT, John Hendrikx wrote: > BitSet uses the SetListenerHelper abstraction to prevent allocating the > listener arrays. > > When removing listeners, the newly returned listener helper (which may be > different from the one called) is not reassigned. This effectively

Re: RFR: JDK-8304933: BitSet (used for CSS pseudo class states) listener management is incorrect

2023-04-10 Thread Kevin Rushforth
On Mon, 3 Apr 2023 07:23:28 GMT, John Hendrikx wrote: >> modules/javafx.graphics/src/main/java/com/sun/javafx/css/BitSet.java line >> 603: >> >>> 601: @Override >>> 602: public void removeListener(SetChangeListener >>> setChangeListener) { >>> 603: if (setChangeListener != null

Re: RFR: 8150709: Mac OSX and German Keyboard Layout (Y/Z) [v7]

2023-04-10 Thread Martin Fox
On Wed, 5 Apr 2023 13:25:00 GMT, Jose Pereda wrote: >> Martin Fox has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Added manual cross-platform keyboard handling test > > tests/manual/events/KeyboardTest.java line 458: > >> 456: G

Re: RFR: 8301312: Create implementation of NSAccessibilityButton protocol

2023-04-10 Thread Kevin Rushforth
On Mon, 10 Apr 2023 18:54:44 GMT, Alexander Zuev wrote: >> modules/javafx.graphics/src/main/java/com/sun/glass/ui/mac/MacAccessible.java >> line 807: >> >>> 805: if (this.peer == 0L) { >>> 806: AccessibleRole role = (AccessibleRole) getAttribute(ROLE); >>> 807: i

Re: RFR: 8150709: Mac OSX and German Keyboard Layout (Y/Z) [v7]

2023-04-10 Thread Martin Fox
On Wed, 5 Apr 2023 12:51:31 GMT, Jose Pereda wrote: >> Martin Fox has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Added manual cross-platform keyboard handling test > > modules/javafx.graphics/src/main/native-glass/mac/GlassKey.m line 47

Re: RFR: 8150709: Mac OSX and German Keyboard Layout (Y/Z) [v7]

2023-04-10 Thread Martin Fox
On Wed, 5 Apr 2023 12:39:28 GMT, Jose Pereda wrote: >> Martin Fox has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Added manual cross-platform keyboard handling test > > modules/javafx.graphics/src/main/native-glass/mac/GlassKey.m line 19

Re: RFR: 8150709: Mac OSX and German Keyboard Layout (Y/Z) [v7]

2023-04-10 Thread Martin Fox
On Wed, 5 Apr 2023 12:36:37 GMT, Jose Pereda wrote: >> Martin Fox has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Added manual cross-platform keyboard handling test > > modules/javafx.graphics/src/main/native-glass/mac/GlassKey.m line 19

Re: RFR: 8150709: Mac OSX and German Keyboard Layout (Y/Z) [v8]

2023-04-10 Thread Martin Fox
> This PR adds code to ensure that KeyCodeCombinations match KeyEvents as > expected by more accurately mapping from a Mac key code to a Java key code > based on the user’s active keyboard layout (the existing code assumes a US > QWERTY layout). The new code first identifies a set of Mac keys wh

Re: RFR: 8301312: Create implementation of NSAccessibilityButton protocol

2023-04-10 Thread Alexander Zuev
On Sat, 8 Apr 2023 14:19:34 GMT, Kevin Rushforth wrote: >> Add the common base component for all the new implementing native classes >> Change native peer creation to use the new base component The new code will >> instantiate new protocol implementation for the given role if it exists or >>

Re: RFR: 8301312: Create implementation of NSAccessibilityButton protocol

2023-04-10 Thread Alexander Zuev
On Sat, 8 Apr 2023 14:18:56 GMT, Kevin Rushforth wrote: >> Add the common base component for all the new implementing native classes >> Change native peer creation to use the new base component The new code will >> instantiate new protocol implementation for the given role if it exists or >>

Re: RFR: JDK-8224260: ChangeListener not triggered when adding a new listener in invalidated method

2023-04-10 Thread Michael Strauß
On Thu, 30 Mar 2023 21:53:48 GMT, John Hendrikx wrote: > Fixes three issues in ExpressionHelper: > > - Current Value was not retained when changing from SingleChange to Generic, > this can lead to missed changes > - Current Value was not retained when changing from Generic to SingleChange, > t

Re: RFR: JDK-8304439: Subscription based listeners [v2]

2023-04-10 Thread Michael Strauß
On Mon, 27 Mar 2023 14:36:45 GMT, John Hendrikx wrote: >> Makes `Subscription` public (removing some of its methods that are >> unnecessary), and adds methods that can provide `Subscription`s in >> `ObservableValue`. > > John Hendrikx has updated the pull request incrementally with one addition

Re: RFR: 8301312: Create implementation of NSAccessibilityButton protocol

2023-04-10 Thread Alexander Zuev
On Sat, 8 Apr 2023 14:16:57 GMT, Kevin Rushforth wrote: >> Add the common base component for all the new implementing native classes >> Change native peer creation to use the new base component The new code will >> instantiate new protocol implementation for the given role if it exists or >>

[jfx20u] RFR: 8286089: Intermittent WebKit build failure on macOS in JavaScriptCore

2023-04-10 Thread Jay Bhaskar
clean backport to jfx20u. The fix solves the build issue on the macOx11_64 machine. It fixes the intermittent build issue. The build issue occurs without the fix and does not occur with the fix. - Commit messages: - Backport 810bd90d5089364067549b8ba299300bfd208357 Changes: https:

[jfx20u] RFR: 8282359: Intermittent WebKit build failure on Windows: C1090: PDB API call failed, error code 23

2023-04-10 Thread Hima Bindu Meda
Clean Backport.This pull request contains a backport of commit [a264435d](https://github.com/openjdk/jfx/commit/a264435dccba6ec386548f76f1ace095d943f4ca) from the [openjdk/jfx](https://git.openjdk.org/jfx) repository. - Commit messages: - Backport a264435dccba6ec386548f76f1ace095d9

Integrated: 8260528: Clean glass-gtk sizing and positioning code

2023-04-10 Thread Thiago Milczarek Sayao
On Thu, 13 Oct 2022 00:48:09 GMT, Thiago Milczarek Sayao wrote: > This cleans size and positioning code, reducing special cases, code > complexity and size. > > Changes: > > - cached extents: 28, 1, 1, 1 are old defaults - modern gnome uses different > sizes. It does not assume any size beca

Proposed schedule for JavaFX 21

2023-04-10 Thread Kevin Rushforth
Here is the proposed schedule for JavaFX 21. RDP1: Jul 13, 2023 (aka “feature freeze”) RDP2: Aug 3, 2023 Freeze: Aug 31, 2023 GA: Sep 19, 2023 We plan to fork a jfx21 stabilization branch at RDP1. The start of RDP1, the start of RDP2, and the code freeze will be 16:00 UTC on the respective dat

Integrated: 8303740 JavaFX - Leak in Logging, Logging remembers last exception

2023-04-10 Thread Florian Kirmaier
On Tue, 7 Mar 2023 11:59:20 GMT, Florian Kirmaier wrote: > When an exception is logged in JavaFX, then the exception is kept in a > reference. > This way, always the last logged exception is retained. > > This is a memory-leak. > This was done to write unit-tests to ensure certain error-cases a

Re: RFR: 8088594: NullPointerException on showing submenu of a contextmenu [v2]

2023-04-10 Thread Karthik P K
> When custom skin was loaded, the listeners added in `ContextMenuContent` > class while loading the default skin were not removed. This was causing the > NPE when outdated listeners were invoked. > > Updated the code to dispose listeners in the `dispose` method of > `ContextMenuSkin` so that w