Re: RFR: 8296919: Make system tests that detect memory leaks more robust by using JMemoryBuddy utility [v4]

2023-05-04 Thread Lukasz Kostyra
On Thu, 4 May 2023 15:30:20 GMT, Andy Goryachev wrote: >> tests/system/src/test/java/test/javafx/embed/swing/SwingNodeMemoryLeakTest.java >> line 80: >> >>> 78: }); >>> 79: >>> 80: SwingUtilities.invokeAndWait(() -> Util.sleep(500)); >> >> Sleeping on either the JavaFX Applica

Re: RFR: JDK-8306990: The guarantees given by Region's floor and ceiling functions should work for larger values [v5]

2023-05-04 Thread Andy Goryachev
On Thu, 4 May 2023 09:46:28 GMT, John Hendrikx wrote: >> Region has floor and ceiling functions that ensure that calling them twice >> in a row will yield the same result: >> >> ceil(x) = ceil(ceil(x)) >> >> However, due to use of a constant `EPSILON` which is added/subtracted before >>

RFR: 8307363: TextFlow.underlineShape()

2023-05-04 Thread Andy Goryachev
Adding TextFlow.underlineShape() to add support for a spellchecker-like decoration, using getRange(start, end, TextLayout.TYPE_UNDERLINE); which mirrors an existing method in Text with exactly the same signature. - Commit messages: - use the same javadoc as in Text - whitespace

Re: RFR: JDK-8245919: Region#padding property rendering error [v3]

2023-05-04 Thread Michael Strauß
On Tue, 18 Apr 2023 12:17:53 GMT, John Hendrikx wrote: >> Fix bug in CSS caching code that could reset values on unrelated nodes. >> >> The bug occurs due to a cache entry being constructed incorrectly when the >> initial node that triggered the cache entry creation has user set values. >> The

RFR: 8089373: Translation from character to key code is not sufficient

2023-05-04 Thread Martin Fox
Note: the Java-side changes in this PR are also in #694 which fixes the same issue (and more) on Linux. Unfortunately the Linux Robot code is not working making it difficult to test on that platform (see #718). KeyCharacterCombinations allow the specification of accelerators based on characters

Re: RFR: 8296919: Make system tests that detect memory leaks more robust by using JMemoryBuddy utility [v4]

2023-05-04 Thread Andy Goryachev
On Thu, 4 May 2023 12:05:45 GMT, Kevin Rushforth wrote: >> Lukasz Kostyra has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Update javafx.web classpath to include JMemoryBuddy > > tests/system/src/test/java/test/javafx/embed/swing/SwingNod

Re: RFR: JDK-8306990: The guarantees given by Region's floor and ceiling functions should work for larger values [v5]

2023-05-04 Thread Andy Goryachev
On Thu, 4 May 2023 09:47:02 GMT, John Hendrikx wrote: > I think it was important to track down the root cause here You are so right! I'll try to take a look today. Without looking at the code, could we add a test (unless already done) which exercise all the possible corner cases with Double -

Re: RFR: JDK-8298104: NPE on synchronizeSceneNodes()

2023-05-04 Thread John Hendrikx
On Thu, 4 May 2023 12:11:13 GMT, buedi wrote: > Hello John, this issue occurs very rarely, approximately 1-2 times per month, > on systems that run 24/7. However, when it does occur, the JavaFx application > gets stuck in an endless loop (--> > [logError-JavaFxScene.txt](https://github.com/ope

Withdrawn: 8299348: Size-restricted window can be observed with incorrect dimensions

2023-05-04 Thread duke
On Mon, 26 Dec 2022 03:03:50 GMT, Michael Strauß wrote: > On Windows, a `Stage` that is restricted by minimum and maximum sizes can > briefly be observed to appear with incorrect dimensions when it is first > shown. The root cause of this bug is that the native `WinWindow._setBounds` > method

Re: RFR: 8296919: Make system tests that detect memory leaks more robust by using JMemoryBuddy utility [v4]

2023-05-04 Thread Kevin Rushforth
On Thu, 4 May 2023 08:21:15 GMT, Lukasz Kostyra wrote: >> Verified these changes on all platforms and saw no abnormalities in test >> execution. >> >> This change is based on a preliminary patch done by @aghaisas , with some >> minor changes and the addition of one Leak-related web test. >> E

Re: RFR: JDK-8298104: NPE on synchronizeSceneNodes()

2023-05-04 Thread buedi
On Wed, 3 May 2023 11:46:50 GMT, buedi wrote: > A null pointer exception occurs on the ScenePulseListener when iterating > through the dirty node list. > A null check is needed on the node before calling node.getScene(). > > The error occurs occasionally and causes the application to crash. He

Re: RFR: JDK-8298104: NPE on synchronizeSceneNodes()

2023-05-04 Thread John Hendrikx
On Wed, 3 May 2023 11:46:50 GMT, buedi wrote: > A null pointer exception occurs on the ScenePulseListener when iterating > through the dirty node list. > A null check is needed on the node before calling node.getScene(). > > The error occurs occasionally and causes the application to crash. Hi

Re: RFR: JDK-8306490: Fix raw type warnings in graphics

2023-05-04 Thread Lukasz Kostyra
On Tue, 18 Apr 2023 16:17:52 GMT, John Hendrikx wrote: > Focused only on raw type problems, and removing casts that were no longer > needed because of the changes. Patch seems okay for me, I also run it on Windows and macOS and tests are all good. I left a minor comment about some leftover im

Re: RFR: 8307208: Add GridPane constructor that accepts hGap and vGap values [v2]

2023-05-04 Thread John Hendrikx
On Wed, 3 May 2023 21:22:10 GMT, Marius Hanl wrote: >> This PR adds a new constructor to `GridPane` that accepts a `hgap` and a >> `vgap`. >> As also written in the ticket, this is a small enhancement to improve the >> creation of a layout involving a `GridPane` without FXML. >> In my experienc

Re: RFR: JDK-8306990: The guarantees given by Region's floor and ceiling functions should work for larger values [v5]

2023-05-04 Thread John Hendrikx
On Thu, 27 Apr 2023 15:23:33 GMT, Andy Goryachev wrote: >> John Hendrikx has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Fix NaN returns when using Math.ulp > > modules/javafx.graphics/src/main/java/javafx/scene/layout/Region.java line

Re: RFR: JDK-8306990: The guarantees given by Region's floor and ceiling functions should work for larger values [v5]

2023-05-04 Thread John Hendrikx
On Thu, 4 May 2023 09:46:28 GMT, John Hendrikx wrote: >> Region has floor and ceiling functions that ensure that calling them twice >> in a row will yield the same result: >> >> ceil(x) = ceil(ceil(x)) >> >> However, due to use of a constant `EPSILON` which is added/subtracted before >>

Re: RFR: JDK-8306990: The guarantees given by Region's floor and ceiling functions should work for larger values [v5]

2023-05-04 Thread John Hendrikx
> Region has floor and ceiling functions that ensure that calling them twice in > a row will yield the same result: > > ceil(x) = ceil(ceil(x)) > > However, due to use of a constant `EPSILON` which is added/subtracted before > doing the rounding, this only works for small numbers (in the r

Re: RFR: 8296919: Make system tests that detect memory leaks more robust by using JMemoryBuddy utility [v4]

2023-05-04 Thread Lukasz Kostyra
> Verified these changes on all platforms and saw no abnormalities in test > execution. > > This change is based on a preliminary patch done by @aghaisas , with some > minor changes and the addition of one Leak-related web test. > EventListenerLeak test was not touched, as it is a separate manu

Re: RFR: 8296919: Make system tests that detect memory leaks more robust by using JMemoryBuddy utility [v3]

2023-05-04 Thread Lukasz Kostyra
On Wed, 3 May 2023 09:20:14 GMT, Lukasz Kostyra wrote: >> Verified these changes on all platforms and saw no abnormalities in test >> execution. >> >> This change is based on a preliminary patch done by @aghaisas , with some >> minor changes and the addition of one Leak-related web test. >> E