Re: RFR: 8274274: Update JUnit to version 5.8.1 [v9]

2021-12-10 Thread Johan Vos
On Mon, 22 Nov 2021 13:43:34 GMT, John Hendrikx wrote: >> I've added JUnit 5 as a test dependency and made sure that the JUnit 4 tests >> still work. Also added a single JUnit 5 tests, and confirmed it works. >> >> I've updated the Eclipse project file for the base module only. > > John

Re: RFR: 8276167: VirtualFlow.scrollToTop doesn't scroll to the top of the last element [v4]

2021-12-10 Thread Johan Vos
On Fri, 10 Dec 2021 15:24:44 GMT, eduardsdv wrote: >> Fix VirtualFlow.scrollToTop(int) doesn't scroll to the top of the last >> element but to the bottom of the last element. > > eduardsdv has updated the pull request incrementally with one additional > commit since the last revision: > >

Re: RFR: 8273096: Add support for H.265/HEVC to JavaFX Media [v2]

2021-12-10 Thread Alexander Matveev
On Tue, 16 Nov 2021 02:24:11 GMT, Alexander Matveev wrote: >> - Added support for H.265/HEVC for all 3 platforms. >> - Support is added only for .mp4 files over FILE/HTTP/HTTPS protocols. HTTP >> Live Streaming with H.265/HEVC is not supported. >> - On Windows mfwrapper was introduced which

Re: RFR: 8273096: Add support for H.265/HEVC to JavaFX Media [v2]

2021-12-10 Thread Alexander Matveev
On Fri, 10 Dec 2021 19:34:25 GMT, Kevin Rushforth wrote: >> Alexander Matveev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8273096: Add support for H.265/HEVC to JavaFX Media [v3] > >

Re: RFR: 8273096: Add support for H.265/HEVC to JavaFX Media [v3]

2021-12-10 Thread Alexander Matveev
> - Added support for H.265/HEVC for all 3 platforms. > - Support is added only for .mp4 files over FILE/HTTP/HTTPS protocols. HTTP > Live Streaming with H.265/HEVC is not supported. > - On Windows mfwrapper was introduced which uses Media Foundation APIs to > decode HEVC. > - 10 and 12-bit

Re: RFR: 8271054: [REDO] Wrong stage gets focused after modal stage creation [v7]

2021-12-10 Thread Kevin Rushforth
On Mon, 1 Nov 2021 16:46:34 GMT, Thiago Milczarek Sayao wrote: >> Found the problem thru this path: >> >> **WindowStage.java** >> >> final void handleFocusDisabled() { >> if (activeWindows.isEmpty()) { >> return; >> } >> WindowStage window =

[jfx17u] Integrated: 8272638: Update copyright header for files modified in 2021

2021-12-10 Thread Kevin Rushforth
On Tue, 7 Dec 2021 13:30:44 GMT, Kevin Rushforth wrote: > Backport fix to update copyright changes. This was done by a script that > checks the copyright years for all files modified this year and updates the > ones that are missing 2021. As such it won't be a clean backport. This pull

[jfx11u] Integrated: 8272638: Update copyright header for files modified in 2021

2021-12-10 Thread Kevin Rushforth
On Tue, 7 Dec 2021 13:31:38 GMT, Kevin Rushforth wrote: > Backport fix to update copyright headers. This was done by a script that > checks the copyright years for all files modified this year and updates the > ones that are missing 2021. As such it won't be a clean backport. This pull

Re: [jfx17u] RFR: 8272638: Update copyright header for files modified in 2021

2021-12-10 Thread Ambarish Rapte
On Tue, 7 Dec 2021 13:30:44 GMT, Kevin Rushforth wrote: > Backport fix to update copyright changes. This was done by a script that > checks the copyright years for all files modified this year and updates the > ones that are missing 2021. As such it won't be a clean backport. Marked as

Re: [jfx11u] RFR: 8272638: Update copyright header for files modified in 2021

2021-12-10 Thread Ambarish Rapte
On Tue, 7 Dec 2021 13:31:38 GMT, Kevin Rushforth wrote: > Backport fix to update copyright headers. This was done by a script that > checks the copyright years for all files modified this year and updates the > ones that are missing 2021. As such it won't be a clean backport. Marked as

Re: RFR: 8276167: VirtualFlow.scrollToTop doesn't scroll to the top of the last element [v3]

2021-12-10 Thread Kevin Rushforth
On Fri, 10 Dec 2021 14:58:25 GMT, Johan Vos wrote: >> This part was originally copied from the 'VirtualContainerBase'. There was >> this comment: _special-case the 'greater than row count' condition to have >> it be perfectly at position 1_. >> >> Maybe to avoid throwing the

Re: RFR: 8278494: Remove .hgtags

2021-12-10 Thread Ambarish Rapte
On Thu, 9 Dec 2021 13:02:32 GMT, Kevin Rushforth wrote: > Trivial fix to remove the obsolete `.hgtags` file from the repo. As mentioned > in JBS, this file has (intentionally) not been updated since the switch to > git. It will continue to be available in the `jfx17` branch >

Integrated: 8191995: Regression: DatePicker must commit on focusLost

2021-12-10 Thread Marius Hanl
On Wed, 24 Nov 2021 09:09:53 GMT, Marius Hanl wrote: > This PR fixes an issue where the `DatePicker` is not committing his text as > value when the focus is lost. > As the ticket also mentions, this is a regression which last worked on JavaFX > 8 and got broken by this refactoring: >

Re: RFR: 8268225: Support :focus-visible and :focus-within CSS pseudoclasses [v7]

2021-12-10 Thread Jeanette Winzenburg
On Tue, 7 Dec 2021 15:44:26 GMT, Michael Strauß wrote: >However, I'd still argue that a node that is not part of the scene graph >(non-atomically) should not be the focus owner of the scene graph. I agree :) But that's what seems to happen: in my example, add a handler to remove the focused

Re: RFR: 8234921: Add DirectionalLight to the selection of 3D light types [v2]

2021-12-10 Thread Kevin Rushforth
On Sat, 14 Aug 2021 13:00:52 GMT, Nir Lisker wrote: >> Adds a directional light as a subclass of `LightBase`. I think that this is >> the correct hierarchy for it. >> >> I tried to simulate a directional light by putting a point light far away, >> but I got artifacts when the distance was

Re: RFR: 8276167: VirtualFlow.scrollToTop doesn't scroll to the top of the last element [v3]

2021-12-10 Thread eduardsdv
On Mon, 6 Dec 2021 14:58:52 GMT, eduardsdv wrote: >> Fix VirtualFlow.scrollToTop(int) doesn't scroll to the top of the last >> element but to the bottom of the last element. > > eduardsdv has updated the pull request incrementally with one additional > commit since the last revision: > >

RFR: 8278494: Remove .hgtags

2021-12-10 Thread Kevin Rushforth
Trivial fix to remove the obsolete `.hgtags` file from the repo. As mentioned in JBS, this file has (intentionally) not been updated since the switch to git. It will continue to be available in the `jfx17` branch [here](https://github.com/openjdk/jfx/blob/jfx17/.hgtags). The corresponding file

RFR: 8278134: Move static utility methods to infrastructure (EditAndScrollTest)

2021-12-10 Thread Jeanette Winzenburg
Extracted static test utility methods from EditAndScrollTest into new VirtualizedControlTestUtils, added rudimentary tests for the methods. - Commit messages: - 8278134: Move static utility methods to infrastructure Changes: https://git.openjdk.java.net/jfx/pull/690/files Webrev:

Re: RFR: 8277848 Binding and Unbinding to List leads to memory leak [v2]

2021-12-10 Thread Florian Kirmaier
> Making the initial listener of the ListProperty weak fixes the problem. > The same is fixed for Set and Map. > Due to a smart implementation, this is done without any performance drawback. > (The trick is to have an object, which is both the WeakReference and the > Changelistener) > By implying

Re: RFR: 8269907 memory leak - Dirty Nodes / Parent removed [v4]

2021-12-10 Thread Florian Kirmaier
On Thu, 9 Dec 2021 12:45:32 GMT, Florian Kirmaier wrote: >> After thinking about this issue for some time, I've now got a solution. >> I know put the scene in the state it is, before is was shown, when the >> dirtyNodes are unset, the whole scene is basically considered dirty. >> This has the

Re: RFR: 8269907 memory leak - Dirty Nodes / Parent removed [v4]

2021-12-10 Thread Florian Kirmaier
> After thinking about this issue for some time, I've now got a solution. > I know put the scene in the state it is, before is was shown, when the > dirtyNodes are unset, the whole scene is basically considered dirty. > This has the drawback of rerendering, whenever a window is "reshown", but it

Re: RFR: 8273096: Add support for H.265/HEVC to JavaFX Media [v2]

2021-12-10 Thread Kevin Rushforth
On Tue, 16 Nov 2021 02:24:11 GMT, Alexander Matveev wrote: >> - Added support for H.265/HEVC for all 3 platforms. >> - Support is added only for .mp4 files over FILE/HTTP/HTTPS protocols. HTTP >> Live Streaming with H.265/HEVC is not supported. >> - On Windows mfwrapper was introduced which

Re: RFR: 8274967: KeyCharacterCombinations for punctuation and symbols fail on non-US keyboards

2021-12-10 Thread Martin Fox
On Mon, 15 Nov 2021 20:13:11 GMT, Martin Fox wrote: > The code that maps from a Windows virtual key code to a Java key code in > `getKeyCodeForChar` did not match the similar code in > `ViewContainer::HandleViewKeyEvent`. The OEM keys are assigned to printable > punctuation and symbols in

Re: RFR: 8276167: VirtualFlow.scrollToTop doesn't scroll to the top of the last element [v4]

2021-12-10 Thread eduardsdv
> Fix VirtualFlow.scrollToTop(int) doesn't scroll to the top of the last > element but to the bottom of the last element. eduardsdv has updated the pull request incrementally with one additional commit since the last revision: 8276167: Avoid additional Toolkit.getToolkit().firePulse()

Re: RFR: 8276167: VirtualFlow.scrollToTop doesn't scroll to the top of the last element [v3]

2021-12-10 Thread eduardsdv
On Fri, 10 Dec 2021 15:00:56 GMT, Johan Vos wrote: >> Without this line the test fails in the line 1865 with the message >> 'expected:<7> but was:<8>'. >> I think this is because in the line 1850 'listView.scrollTo(99)' is >> executed, which now does not set the position to 1. >> >> It can

Integrated: 8201538: Remove implementation support for applets from JavaFX

2021-12-10 Thread Kevin Rushforth
On Tue, 31 Aug 2021 16:28:53 GMT, Kevin Rushforth wrote: > This PR removes the obsolete applet implementation from JavaFX. It is an > ongoing maintenance burden to carry around this legacy code. Also, cleaning > this up could help in the implementation of GTK4, Wayland, and Metal, since > we

Re: RFR: 8276167: VirtualFlow.scrollToTop doesn't scroll to the top of the last element [v3]

2021-12-10 Thread Johan Vos
On Fri, 10 Dec 2021 13:34:53 GMT, eduardsdv wrote: >> modules/javafx.controls/src/main/java/javafx/scene/control/skin/VirtualFlow.java >> line 1584: >> >>> 1582: boolean posSet = false; >>> 1583: >>> 1584: if (index > getCellCount() - 1) { >> >> I agree that the previous code

Re: RFR: 8201538: Remove implementation support for applets from JavaFX

2021-12-10 Thread Johan Vos
On Tue, 31 Aug 2021 16:28:53 GMT, Kevin Rushforth wrote: > This PR removes the obsolete applet implementation from JavaFX. It is an > ongoing maintenance burden to carry around this legacy code. Also, cleaning > this up could help in the implementation of GTK4, Wayland, and Metal, since > we

Integrated: 8278494: Remove .hgtags

2021-12-10 Thread Kevin Rushforth
On Thu, 9 Dec 2021 13:02:32 GMT, Kevin Rushforth wrote: > Trivial fix to remove the obsolete `.hgtags` file from the repo. As mentioned > in JBS, this file has (intentionally) not been updated since the switch to > git. It will continue to be available in the `jfx17` branch >

Re: RFR: 8201538: Remove implementation support for applets from JavaFX

2021-12-10 Thread Kevin Rushforth
On Tue, 31 Aug 2021 16:28:53 GMT, Kevin Rushforth wrote: > This PR removes the obsolete applet implementation from JavaFX. It is an > ongoing maintenance burden to carry around this legacy code. Also, cleaning > this up could help in the implementation of GTK4, Wayland, and Metal, since > we

Integrated: 8278425: TreeTableCellStartEditTest uses deprecated TreeTableCell methods

2021-12-10 Thread Marius Hanl
On Fri, 10 Dec 2021 00:12:20 GMT, Marius Hanl wrote: > This PR replaces 2 deprecations which were added in > https://bugs.openjdk.java.net/browse/JDK-8188026 This pull request has now been integrated. Changeset: 79151937 Author:Marius Hanl Committer: Kevin Rushforth URL:

Re: RFR: 8276167: VirtualFlow.scrollToTop doesn't scroll to the top of the last element [v3]

2021-12-10 Thread eduardsdv
On Fri, 10 Dec 2021 12:33:29 GMT, Johan Vos wrote: >> eduardsdv has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 8276170: Add junit for VirtualFlow.scrollToTop(int) > >

Re: RFR: 8276167: VirtualFlow.scrollToTop doesn't scroll to the top of the last element [v3]

2021-12-10 Thread eduardsdv
On Fri, 10 Dec 2021 12:33:19 GMT, Johan Vos wrote: >> eduardsdv has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 8276170: Add junit for VirtualFlow.scrollToTop(int) > >

Re: RFR: 8278425: TreeTableCellStartEditTest uses deprecated TreeTableCell methods

2021-12-10 Thread Kevin Rushforth
On Fri, 10 Dec 2021 00:12:20 GMT, Marius Hanl wrote: > This PR replaces 2 deprecations which were added in > https://bugs.openjdk.java.net/browse/JDK-8188026 Marked as reviewed by kcr (Lead). - PR: https://git.openjdk.java.net/jfx/pull/692

Re: RFR: 8276167: VirtualFlow.scrollToTop doesn't scroll to the top of the last element [v3]

2021-12-10 Thread Johan Vos
On Mon, 6 Dec 2021 14:58:52 GMT, eduardsdv wrote: >> Fix VirtualFlow.scrollToTop(int) doesn't scroll to the top of the last >> element but to the bottom of the last element. > > eduardsdv has updated the pull request incrementally with one additional > commit since the last revision: > >

Re: RFR: 8276167: VirtualFlow.scrollToTop doesn't scroll to the top of the last element [v3]

2021-12-10 Thread Ajit Ghaisas
On Mon, 6 Dec 2021 14:58:52 GMT, eduardsdv wrote: >> Fix VirtualFlow.scrollToTop(int) doesn't scroll to the top of the last >> element but to the bottom of the last element. > > eduardsdv has updated the pull request incrementally with one additional > commit since the last revision: > >

Re: RFR: 8274967: KeyCharacterCombinations for punctuation and symbols fail on non-US keyboards

2021-12-10 Thread Pankaj Bansal
On Mon, 15 Nov 2021 20:13:11 GMT, Martin Fox wrote: > The code that maps from a Windows virtual key code to a Java key code in > `getKeyCodeForChar` did not match the similar code in > `ViewContainer::HandleViewKeyEvent`. The OEM keys are assigned to printable > punctuation and symbols in

Re: RFR: 8201538: Remove implementation support for applets from JavaFX

2021-12-10 Thread Ajit Ghaisas
On Tue, 31 Aug 2021 16:28:53 GMT, Kevin Rushforth wrote: > This PR removes the obsolete applet implementation from JavaFX. It is an > ongoing maintenance burden to carry around this legacy code. Also, cleaning > this up could help in the implementation of GTK4, Wayland, and Metal, since > we