Re: RFR: 8092102: Labeled: truncated property [v9]

2024-04-30 Thread Ambarish Rapte
On Wed, 10 Apr 2024 21:25:10 GMT, Andy Goryachev wrote: >> Adds **Labeled.textTruncated** property which indicates when the text is >> visually truncated (and the ellipsis string is inserted) in order to fit the >> available width. >> >> The new property reacts to changes in the following prop

Re: RFR: 8330462: StringIndexOutOfBoundException when typing anything into TextField [v24]

2024-04-29 Thread Ambarish Rapte
On Mon, 29 Apr 2024 15:47:54 GMT, Oliver Kopp wrote: >> modules/javafx.graphics/src/main/java/com/sun/glass/ui/win/WinTextRangeProvider.java >> line 104: >> >>> 102: int length = text.length(); >>> 103: start = Utils.clamp(0, start, length); >>> 104: end = Utils.clamp(st

Re: RFR: 8330462: StringIndexOutOfBoundException when typing anything into TextField [v24]

2024-04-29 Thread Ambarish Rapte
On Fri, 26 Apr 2024 22:58:37 GMT, Oliver Kopp wrote: >> Fixes https://bugs.openjdk.org/browse/JDK-8330462. >> >> If the parameter `maxLength` is larger than `Integer.MAX_VALUE - start`, >> then an addition of `start` to it leads to a negative value. This is "fixed" >> by using `Math.max` compa

Re: RFR: 8330462: StringIndexOutOfBoundException when typing anything into TextField [v24]

2024-04-29 Thread Ambarish Rapte
On Fri, 26 Apr 2024 22:58:37 GMT, Oliver Kopp wrote: >> Fixes https://bugs.openjdk.org/browse/JDK-8330462. >> >> If the parameter `maxLength` is larger than `Integer.MAX_VALUE - start`, >> then an addition of `start` to it leads to a negative value. This is "fixed" >> by using `Math.max` compa

Re: RFR: 8330462: StringIndexOutOfBoundException when typing anything into TextField [v23]

2024-04-26 Thread Ambarish Rapte
On Thu, 25 Apr 2024 21:41:14 GMT, Oliver Kopp wrote: >> Fixes https://bugs.openjdk.org/browse/JDK-8330462. >> >> If the parameter `maxLength` is larger than `Integer.MAX_VALUE - start`, >> then an addition of `start` to it leads to a negative value. This is "fixed" >> by using `Math.max` compa

Re: RFR: 8330462: StringIndexOutOfBoundException when typing anything into TextField [v20]

2024-04-24 Thread Ambarish Rapte
On Wed, 24 Apr 2024 12:54:01 GMT, Oliver Kopp wrote: >> Fixes https://bugs.openjdk.org/browse/JDK-8330462. >> >> If the parameter `maxLength` is larger than `Integer.MAX_VALUE - start`, >> then an addition of `start` to it leads to a negative value. This is "fixed" >> by using `Math.max` compa

Re: RFR: 8330462: StringIndexOutOfBoundException when typing anything into TextField [v20]

2024-04-24 Thread Ambarish Rapte
On Wed, 24 Apr 2024 12:54:01 GMT, Oliver Kopp wrote: >> Fixes https://bugs.openjdk.org/browse/JDK-8330462. >> >> If the parameter `maxLength` is larger than `Integer.MAX_VALUE - start`, >> then an addition of `start` to it leads to a negative value. This is "fixed" >> by using `Math.max` compa

Re: RFR: 8329705: Add missing Application thread checks to platform specific a11y methods

2024-04-05 Thread Ambarish Rapte
On Thu, 4 Apr 2024 21:55:11 GMT, Alexander Zuev wrote: > Added missing checks; > Added a manual regression test; Marked as reviewed by arapte (Reviewer). - PR Review: https://git.openjdk.org/jfx/pull/1436#pullrequestreview-1983111622

Re: [jfx22u] RFR: 8327979: Change JavaFX release version to 22.0.2 in jfx22u

2024-03-12 Thread Ambarish Rapte
On Tue, 12 Mar 2024 13:44:48 GMT, Kevin Rushforth wrote: > Bump version number to 22.0.2 Marked as reviewed by arapte (Reviewer). - PR Review: https://git.openjdk.org/jfx22u/pull/19#pullrequestreview-1931624966

Re: RFR: 8327179: Update the 3D lighting application

2024-03-06 Thread Ambarish Rapte
On Sun, 3 Mar 2024 22:29:02 GMT, Nir Lisker wrote: > Update for the 3D lighting test tool as described in the JBS issue. An observation out of scope of this PR. The effect of lighting on mesh vs a box is quite different. Notice the screenshot, left side is a Box and right side is Mesh, both hav

Re: RFR: 8327179: Update the 3D lighting application

2024-03-06 Thread Ambarish Rapte
On Sun, 3 Mar 2024 22:29:02 GMT, Nir Lisker wrote: > Update for the 3D lighting test tool as described in the JBS issue. providing few more comments tests/performance/3DLighting/src/main/java/lighting3D/Benchmark.java line 49: > 47: stopGraphic.setBoundsType(TextBoundsType.VISUAL); > 4

Re: RFR: 8327179: Update the 3D lighting application

2024-03-05 Thread Ambarish Rapte
On Sun, 3 Mar 2024 22:29:02 GMT, Nir Lisker wrote: > Update for the 3D lighting test tool as described in the JBS issue. Couple quick comments: - Executing the LightingSample manually fails with an exception Caused by: java.lang.NullPointerException: Input stream must not be null at ja

Re: RFR: 8325073: javadoc warnings: missing @param tags and other issues [v3]

2024-03-01 Thread Ambarish Rapte
On Fri, 1 Mar 2024 15:53:21 GMT, Andy Goryachev wrote: >> This change brings the number of javadoc warnings back to 91 (to be fixed in >> [JDK-8270996](https://bugs.openjdk.org/browse/JDK-8270996)) >> >> - adds missing information in `@param` tags >> - adds `@SuppressWarnings("doclint:missing"

Re: RFR: 8325073: javadoc warnings: missing @param tags and other issues [v2]

2024-02-29 Thread Ambarish Rapte
On Fri, 1 Mar 2024 00:22:05 GMT, Andy Goryachev wrote: >> This change brings the number of javadoc warnings back to 91 (to be fixed in >> [JDK-8270996](https://bugs.openjdk.org/browse/JDK-8270996)) >> >> - adds missing information in `@param` tags >> - adds `@SuppressWarnings("doclint:missing"

Re: RFR: 8324233: Update JPEG Image Decoding Software to 9f [v3]

2024-02-29 Thread Ambarish Rapte
On Thu, 29 Feb 2024 09:28:04 GMT, Jayathirth D V wrote: >> IJG has released latest version of libjpeg 9f and we need to update our >> version also match 9f changes. >> IJG reference : https://www.ijg.org/ >> >> With updated changes both headless and headful tests are green on all >> platforms.

Re: RFR: 8324233: Update JPEG Image Decoding Software to 9f [v2]

2024-02-29 Thread Ambarish Rapte
On Wed, 28 Feb 2024 05:19:06 GMT, Jayathirth D V wrote: >> IJG has released latest version of libjpeg 9f and we need to update our >> version also match 9f changes. >> IJG reference : https://www.ijg.org/ >> >> With updated changes both headless and headful tests are green on all >> platforms.

Re: RFR: 8314147: Updated the PhongMaterial documentation [v10]

2024-02-28 Thread Ambarish Rapte
On Wed, 28 Feb 2024 18:48:21 GMT, Nir Lisker wrote: >> Overhaul to the `PhongMaterial` documentation (and a bit to its superclass >> `Material`). Except for the introduction, I divided the documentation into 3 >> sections: qualitative explanation, mathematical model (I wouldn't think it >> nec

Re: RFR: 8314147: Updated the PhongMaterial documentation [v6]

2024-02-27 Thread Ambarish Rapte
On Mon, 26 Feb 2024 16:41:05 GMT, Nir Lisker wrote: >> Overhaul to the `PhongMaterial` documentation (and a bit to its superclass >> `Material`). Except for the introduction, I divided the documentation into 3 >> sections: qualitative explanation, mathematical model (I wouldn't think it >> nec

Re: RFR: 8314147: Updated the PhongMaterial documentation [v6]

2024-02-27 Thread Ambarish Rapte
On Mon, 26 Feb 2024 16:41:05 GMT, Nir Lisker wrote: >> Overhaul to the `PhongMaterial` documentation (and a bit to its superclass >> `Material`). Except for the introduction, I divided the documentation into 3 >> sections: qualitative explanation, mathematical model (I wouldn't think it >> nec

Integrated: 8309374: Accessibility Focus Rectangle on ListItem is not drawn when ListView is shown for first time

2024-02-15 Thread Ambarish Rapte
On Tue, 13 Feb 2024 12:29:49 GMT, Ambarish Rapte wrote: > This is accessibility specific fix. > > **Issue**: When a ListView is shown for first time then accessibility focus > rectangle is not drawn around the focused ListIem > > **Cause:** >The ListView takes a

Re: RFR: 8309374: Accessibility Focus Rectangle on ListItem is not drawn when ListView is shown for first time [v2]

2024-02-15 Thread Ambarish Rapte
On Thu, 15 Feb 2024 21:59:18 GMT, Alexander Zuev wrote: >> Ambarish Rapte has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review corrections-1 > > modules/javafx.controls/src/main/java/javafx/scene/co

Re: RFR: 8309374: Accessibility Focus Rectangle on ListItem is not drawn when ListView is shown for first time [v2]

2024-02-15 Thread Ambarish Rapte
On Thu, 15 Feb 2024 09:57:17 GMT, Marius Hanl wrote: >> Ambarish Rapte has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review corrections-1 > > modules/javafx.controls/src/main/java/javafx/scene/con

Re: [jfx22] RFR: 8325873: Update JDK_DOCS property to point to JDK 21 docs

2024-02-14 Thread Ambarish Rapte
On Thu, 15 Feb 2024 00:31:23 GMT, Kevin Rushforth wrote: > Clean backport of a docs-only fix to jfx22. Marked as reviewed by arapte (Reviewer). - PR Review: https://git.openjdk.org/jfx/pull/1369#pullrequestreview-1881602522

Re: RFR: 8325873: Update JDK_DOCS property to point to JDK 21 docs

2024-02-14 Thread Ambarish Rapte
On Wed, 14 Feb 2024 15:19:47 GMT, Kevin Rushforth wrote: > Simple fix to update the pointer to the JDK docs to JDK 21 (matching the > current boot JDK). The only impact of this change is that hyperlinks in the > generated API docs will link to the online JDK 21 docs. As this is a > docs-only c

Re: RFR: 8309374: Accessibility Focus Rectangle on ListItem is not drawn when ListView is shown for first time [v2]

2024-02-13 Thread Ambarish Rapte
On Tue, 13 Feb 2024 21:15:16 GMT, Kevin Rushforth wrote: > The fix looks good to me, and my initial testing shows that it works as > expected. I left a few suggestions on minor wording changes inline. I'll > reapprove if you make the changes. Thank you Kevin, Please have a re-look, I made the

Re: RFR: 8309374: Accessibility Focus Rectangle on ListItem is not drawn when ListView is shown for first time [v2]

2024-02-13 Thread Ambarish Rapte
K-8309374) > Move through the TextFields and press `Alt+down`, observe that focus > rectangle is drawn correctly. > Once the ListView is showing Press and hold `Up/Down` or `Ctrl + > Up/Down` keys, observe that focus rectangle is always drawn. Ambarish Rapte has updated the p

RFR: 8309374: Accessibility Focus Rectangle on ListItem is not drawn when ListView is shown for first time

2024-02-13 Thread Ambarish Rapte
This is accessibility specific fix. **Issue**: When a ListView is shown for first time then accessibility focus rectangle is not drawn around the focused ListIem **Cause:** The ListView takes a little time to create it's skin(ListViewSkin) and the skins for ListItems(ListCellSkin) If the

Re: RFR: 8307117: TextArea: wrapText property ignored when changing font [v3]

2024-02-11 Thread Ambarish Rapte
On Wed, 7 Feb 2024 21:41:10 GMT, Andy Goryachev wrote: >> Requesting content layout when font changes. >> >> This change makes the visual impact of >> [JDK-8314683](https://bugs.openjdk.org/browse/JDK-8314683) more visible, so >> perhaps both bugs should be fixed at the same time. > > Andy Gor

Re: RFR: 8307117: TextArea: wrapText property ignored when changing font

2024-02-07 Thread Ambarish Rapte
On Mon, 21 Aug 2023 21:44:12 GMT, Andy Goryachev wrote: > Requesting content layout when font changes. > > This change makes the visual impact of > [JDK-8314683](https://bugs.openjdk.org/browse/JDK-8314683) more visible, so > perhaps both bugs should be fixed at the same time. With this chang

Re: RFR: 8307117: TextArea: wrapText property ignored when changing font

2024-02-07 Thread Ambarish Rapte
On Mon, 21 Aug 2023 21:44:12 GMT, Andy Goryachev wrote: > Requesting content layout when font changes. > > This change makes the visual impact of > [JDK-8314683](https://bugs.openjdk.org/browse/JDK-8314683) more visible, so > perhaps both bugs should be fixed at the same time. modules/javafx.

Re: RFR: 8312603: ArrayIndexOutOfBoundsException in Marlin when scaleX is 0 [v6]

2024-02-06 Thread Ambarish Rapte
On Tue, 6 Feb 2024 13:49:13 GMT, Laurent Bourgès wrote: >> Fixed scale=0 in DMarlinPrismUtils + added new test Scale0Test.java > > Laurent Bourgès has updated the pull request incrementally with one > additional commit since the last revision: > > minor cleanup to simplify test and fixed impo

Re: RFR: 8312603: ArrayIndexOutOfBoundsException in Marlin when scaleX is 0 [v5]

2024-02-06 Thread Ambarish Rapte
On Tue, 6 Feb 2024 00:06:58 GMT, Laurent Bourgès wrote: >> Fixed scale=0 in DMarlinPrismUtils + added new test Scale0Test.java > > Laurent Bourgès has updated the pull request incrementally with one > additional commit since the last revision: > > improved rendez-vous to collect stderr Provi

Re: RFR: 8312603: ArrayIndexOutOfBoundsException in Marlin when scaleX is 0 [v4]

2024-02-04 Thread Ambarish Rapte
On Fri, 2 Feb 2024 13:14:26 GMT, Laurent Bourgès wrote: >> Fixed scale=0 in DMarlinPrismUtils + added new test Scale0Test.java > > Laurent Bourgès has updated the pull request incrementally with one > additional commit since the last revision: > > fixed tests thanks to + copyright years test

Re: RFR: JDK-8323615: PopupControl.skin.setSkin(Skin) fails to call dispose() on discarded Skin

2024-02-01 Thread Ambarish Rapte
On Thu, 11 Jan 2024 20:13:09 GMT, Marius Hanl wrote: > For some reason the `skinProperty` did not allow to set a new skin when it is > the same class as the previous one. > This leads to multiple issues: > 1. When creating a new skin (same class as previous), the skin will likely > install list

Re: RFR: 8312603: ArrayIndexOutOfBoundsException in Marlin when scaleX is 0 [v3]

2024-02-01 Thread Ambarish Rapte
On Mon, 29 Jan 2024 21:29:31 GMT, Laurent Bourgès wrote: >> Fixed scale=0 in DMarlinPrismUtils + added new test Scale0Test.java > > Laurent Bourgès has updated the pull request incrementally with one > additional commit since the last revision: > > fixed copyright years of modified files + fi

Re: RFR: 8312603: ArrayIndexOutOfBoundsException in Marlin when scaleX is 0 [v2]

2024-02-01 Thread Ambarish Rapte
On Mon, 29 Jan 2024 09:44:04 GMT, Karthik P K wrote: >> Laurent Bourgès has updated the pull request incrementally with one >> additional commit since the last revision: >> >> fixed test package > > modules/javafx.graphics/src/main/java/com/sun/marlin/DPathConsumer2D.java > line 2: > >> 1:

Re: RFR: 8312603: ArrayIndexOutOfBoundsException in Marlin when scaleX is 0 [v3]

2024-02-01 Thread Ambarish Rapte
On Mon, 29 Jan 2024 21:29:31 GMT, Laurent Bourgès wrote: >> Fixed scale=0 in DMarlinPrismUtils + added new test Scale0Test.java > > Laurent Bourgès has updated the pull request incrementally with one > additional commit since the last revision: > > fixed copyright years of modified files + fi

Re: RFR: 8322703: Intermittent crash in WebView in a JFXPanel from IME calls on macOS [v3]

2024-01-30 Thread Ambarish Rapte
On Wed, 17 Jan 2024 12:57:22 GMT, Kevin Rushforth wrote: >> As noted in the JBS bug, this is a follow-on to >> [JDK-8196011](https://bugs.openjdk.org/browse/JDK-8196011) that I discovered >> while testing the fix for >> [JDK-8221261](https://bugs.openjdk.org/browse/JDK-8221261) (a deadlock in

Integrated: 8324270: Update boot JDK to 21.0.2

2024-01-25 Thread Ambarish Rapte
On Mon, 22 Jan 2024 13:45:35 GMT, Ambarish Rapte wrote: > JDK 21.0.2 is now released : https://jdk.java.net/21/ > Did a full CI build including Webkit and executed all headful tests. This pull request has now been integrated. Changeset: 9e7e8e1d Author:Ambarish Rapte URL:

Re: RFR: 8308955: MediaPlayer/AudioClip skip data on seek/loop

2024-01-24 Thread Ambarish Rapte
On Tue, 23 Jan 2024 03:13:11 GMT, Alexander Matveev wrote: > This is regression from JDK-8262365. JDK-8262365 introduced support for > hardware pause for audio device. For some reason we will skip ~500 ms of > audio data after such pause. It is not noticeable for large audio files, but > for

RFR: 8324270: Update boot JDK to 21.0.2

2024-01-22 Thread Ambarish Rapte
JDK 21.0.2 is now released : https://jdk.java.net/21/ Did a full CI build including Webkit and executed all headful tests. - Commit messages: - boot jdk 21.0.2 Changes: https://git.openjdk.org/jfx/pull/1345/files Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=1345&range=00 Issu

Re: [jfx22u] RFR: 8323555: Change JavaFX release version to 22.0.1 in jfx22u

2024-01-12 Thread Ambarish Rapte
On Thu, 11 Jan 2024 23:11:12 GMT, Kevin Rushforth wrote: > Updates for the beginning of the 22.0.1 release. Marked as reviewed by arapte (Reviewer). - PR Review: https://git.openjdk.org/jfx22u/pull/1#pullrequestreview-1817662768

Re: RFR: 8189282: JavaFX: Invalid position of candidate pop-up of InputMethod in Hi-DPI on Windows [v6]

2024-01-09 Thread Ambarish Rapte
On Fri, 5 Jan 2024 01:22:59 GMT, Martin Fox wrote: >> When reporting input method candidate position the code in >> GlassViewEventHandler is not applying the platform scale factors. This is >> causing incorrect IM positions to be reported to glass on hi-dpi monitors. >> >> This PR a no-op on M

Re: RFR: 8323209: Change JavaFX release version to 23

2024-01-08 Thread Ambarish Rapte
On Mon, 8 Jan 2024 21:24:43 GMT, Kevin Rushforth wrote: > Bump the version number of JavaFX to 23. I will integrate this to master as > part of forking the jfx22 stabilization branch, which is scheduled for > Thursday, January 11, 2024 at 16:00 UTC. Marked as reviewed by arapte (Reviewer). --

Integrated: 8322953: Update copyright header for files modified in 2023

2024-01-04 Thread Ambarish Rapte
On Thu, 4 Jan 2024 15:31:12 GMT, Ambarish Rapte wrote: > Update the copyright year in files modified in year 2023. This pull request has now been integrated. Changeset: bad94a6c Author: Ambarish Rapte URL: https://git.openjdk.org/jfx/commit/bad94a6c3551a2f299d387aec0ed597238a3c

RFR: 8322953: Update copyright header for files modified in 2023

2024-01-04 Thread Ambarish Rapte
Update the copyright year in files modified in year 2023. - Commit messages: - copyright year 2023 Changes: https://git.openjdk.org/jfx/pull/1317/files Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=1317&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8322953 Stats: 27 li

Re: RFR: 8321638: Update to SWT 4.30

2024-01-03 Thread Ambarish Rapte
On Wed, 3 Jan 2024 18:39:58 GMT, Kevin Rushforth wrote: > This PR updates the version of SWT that we use we use when building and > testing the `javafx.swt` modules to 4.30. > > NOTE: we do not distribute SWT, but only use it during build and (manual) > test. > > We currently use SWT 4.6. In

Re: RFR: 8321626: [testbug] Mark DualWindowTest and ContextMenuNPETest unstable on Linux

2023-12-12 Thread Ambarish Rapte
On Mon, 11 Dec 2023 13:10:55 GMT, Kevin Rushforth wrote: > Until JDK-8321624 and JDK-8321625 are fixed, mark the following two tests as > unstable on Linux, meaning they will not be run as part of our nightly > headful test runs. > > DualWindowTest - [JDK-8321624](https://bugs.openjdk.org/brow

Re: RFR: 8319779: SystemMenu: memory leak due to listener never being removed [v2]

2023-12-11 Thread Ambarish Rapte
On Mon, 20 Nov 2023 08:00:58 GMT, Johan Vos wrote: >> A listener was added but never removed. >> This patch removes the listener when the menu it links to is cleared. Fix >> for https://bugs.openjdk.org/browse/JDK-8319779 > > Johan Vos has updated the pull request incrementally with one addition

Integrated: 8321434: Update Gradle to 8.5

2023-12-11 Thread Ambarish Rapte
On Fri, 8 Dec 2023 00:53:56 GMT, Ambarish Rapte wrote: > Gradle 8.5.0 released on Nov 29, 2023, supports JDK 21. > We need to update Gradle to 8.5 in order to update the boot JDK. > There are not API level changes in Gradle that we need to work on, hence the > change is only Gr

RFR: 8321434: Update Gradle to 8.5

2023-12-07 Thread Ambarish Rapte
Gradle 8.5.0 released on Nov 29, 2023, supports JDK 21. We need to update Gradle to 8.5 in order to update the boot JDK. There are not API level changes in Gradle that we need to work on, hence the change is only Gradle version update. Also, merging the PR with boot jdk update to 21.0.1 b12

Integrated: 8284544: [Win] Name-Property of Spinner cannot be changed

2023-12-05 Thread Ambarish Rapte
On Mon, 20 Nov 2023 05:37:38 GMT, Ambarish Rapte wrote: > Currently we use the value of spinner as it's `UIA_NamePropertyId` when a11y > client application requests for it. > Ideally we should use the text set by `Node.setAccessibleText()` as the > `UIA_NamePropertyId`. >

[jfx21u] Integrated: 8318714: Update copyright header for files modified in 2023

2023-12-04 Thread Ambarish Rapte
On Mon, 4 Dec 2023 06:39:47 GMT, Ambarish Rapte wrote: > Update Copyright year in files modified in year 2023. This pull request has now been integrated. Changeset: a993b200 Author: Ambarish Rapte URL: https://git.openjdk.org/jfx21u/commit/a993b20052ef561bd3b1958b1bd20f99ca562

[jfx21u] RFR: 8318714: Update copyright header for files modified in 2023

2023-12-03 Thread Ambarish Rapte
Update Copyright year in files modified in year 2023. - Commit messages: - backport 8318714 Changes: https://git.openjdk.org/jfx21u/pull/38/files Webrev: https://webrevs.openjdk.org/?repo=jfx21u&pr=38&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8318714 Stats: 44 lines i

Re: RFR: 8284544: [Win] Name-Property of Spinner cannot be changed [v3]

2023-12-03 Thread Ambarish Rapte
On Fri, 1 Dec 2023 23:34:12 GMT, Kevin Rushforth wrote: >> Ambarish Rapte has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Mac a11y change for VALUE_STRING attribute for Spinner > > modules/javafx.graphics/

[jfx21u] Integrated: 8318708: FX: Update copyright year in docs, readme files to 2024

2023-12-03 Thread Ambarish Rapte
On Fri, 1 Dec 2023 10:57:43 GMT, Ambarish Rapte wrote: > A clean backport to jfx21u, to update copyright year in doc files. This pull request has now been integrated. Changeset: d7b5c1bf Author: Ambarish Rapte URL: https://git.openjdk.org/jfx21u/com

[jfx21u] RFR: 8318708: FX: Update copyright year in docs, readme files to 2024

2023-12-01 Thread Ambarish Rapte
A clean backport to jfx21u, to update copyright year in doc files. - Commit messages: - update cpr-year in docs to 2024 Changes: https://git.openjdk.org/jfx21u/pull/36/files Webrev: https://webrevs.openjdk.org/?repo=jfx21u&pr=36&range=00 Issue: https://bugs.openjdk.org/browse/JDK

Re: RFR: 8092272: [D3D 3D] Need a robust 3D states management for texture [v2]

2023-12-01 Thread Ambarish Rapte
On Fri, 10 Nov 2023 23:39:21 GMT, Nir Lisker wrote: >> Moves the filter setting of the samplers from the device parameters >> configuration to the use-site, allowing for dynamic changes in the sampler. >> This PR does internal plumbing work only to bring it close to the ES2 >> pipeline. A foll

Re: RFR: 8092272: [D3D 3D] Need a robust 3D states management for texture [v2]

2023-12-01 Thread Ambarish Rapte
On Wed, 15 Nov 2023 11:39:30 GMT, Lukasz Kostyra wrote: >> 1. Not when it will be promoted to public API. Adding record components >> breaks backwards compatibility, so making this a record will not allow >> adding more configuration later on. What might be possible is making it an >> interfac

Re: RFR: 8313648: JavaFX application continues to show a black screen after graphic card driver crash

2023-11-28 Thread Ambarish Rapte
On Mon, 27 Nov 2023 19:57:30 GMT, Thorsten Fischer wrote: >> I have put this in D3DContext.java (as per customer suggestion). Just >> wondering if I should just reinitialize directly and not wait loop: in >> testLostStateAndReset in D3DContext.java (D3DERR_DEVICEREMOVED is handled >> further d

Re: RFR: 8284544: [Win] Name-Property of Spinner cannot be changed [v3]

2023-11-22 Thread Ambarish Rapte
On Wed, 22 Nov 2023 17:05:36 GMT, Ambarish Rapte wrote: >> Currently we use the value of spinner as it's `UIA_NamePropertyId` when a11y >> client application requests for it. >> Ideally we should use the text set by `Node.setAccessibleText()` as the >> `UIA_

Re: RFR: 8313556: Create implementation of NSAccessibilitySlider protocol [v6]

2023-11-22 Thread Ambarish Rapte
On Wed, 15 Nov 2023 22:00:13 GMT, Alexander Zuev wrote: >> Create implementation for Slider and Stepper accessibility protocols. >> Fix mapping. >> Fix performAction parameter type in declaration. > > Alexander Zuev has updated the pull request incrementally with one additional > commit since th

Re: RFR: 8284544: [Win] Name-Property of Spinner cannot be changed [v3]

2023-11-22 Thread Ambarish Rapte
t then value of Spinner and then if > editable or not > - 2. In application "Accessibility Insights for Windows", you can see the > text set by `setAccessibleText()` for Spinner as the Name property and the > Spinner's value as value property Ambarish Rapte has updated th

Re: RFR: 8313556: Create implementation of NSAccessibilitySlider protocol [v6]

2023-11-21 Thread Ambarish Rapte
On Wed, 15 Nov 2023 22:00:13 GMT, Alexander Zuev wrote: >> Create implementation for Slider and Stepper accessibility protocols. >> Fix mapping. >> Fix performAction parameter type in declaration. > > Alexander Zuev has updated the pull request incrementally with one additional > commit since th

Re: RFR: 8284544: [Win] Name-Property of Spinner cannot be changed [v2]

2023-11-21 Thread Ambarish Rapte
On Tue, 21 Nov 2023 16:32:26 GMT, Kevin Rushforth wrote: >> Ambarish Rapte has updated the pull request incrementally with one >> additional commit since the last revision: >> >> add @since 22 > > modules/javafx.controls/src/main/java/javafx/scene/control/Spinn

Re: RFR: 8284544: [Win] Name-Property of Spinner cannot be changed [v2]

2023-11-21 Thread Ambarish Rapte
t then value of Spinner and then if > editable or not > - 2. In application "Accessibility Insights for Windows", you can see the > text set by `setAccessibleText()` for Spinner as the Name property and the > Spinner's value as value property Ambarish Rapte has updated th

RFR: 8284544: [Win] Name-Property of Spinner cannot be changed

2023-11-19 Thread Ambarish Rapte
Currently we use the value of spinner as it's `UIA_NamePropertyId` when a11y client application requests for it. Ideally we should use the text set by `Node.setAccessibleText()` as the `UIA_NamePropertyId`. For other controls such as Slider, ListView we use the text set by setAccessibleText() AP

Re: RFR: 8319996: Update to GCC 13.2.0 on Linux [v2]

2023-11-16 Thread Ambarish Rapte
On Wed, 15 Nov 2023 12:35:50 GMT, Kevin Rushforth wrote: >> This PR updates the compiler on Linux to GCC 13.2.0 (from 12.2.0) to match >> JDK 22. >> >> I've run headless and headful tests on Ubuntu 16.04, 20.04, and 22.04. > > Kevin Rushforth has updated the pull request with a new target base

Re: RFR: 8319996: Update to GCC 13.2.0 on Linux

2023-11-14 Thread Ambarish Rapte
On Tue, 14 Nov 2023 00:24:30 GMT, Kevin Rushforth wrote: > This PR updates the compiler on Linux to GCC 13.2.0 (from 12.2.0) to match > JDK 22. > > I've run headless and headful tests on Ubuntu 16.04, 20.04, and 22.04. Marked as reviewed by arapte (Reviewer). - PR Review: https:/

Re: RFR: 8319762: Update to Visual Studio 2022 version 17.6.5 on Windows [v2]

2023-11-14 Thread Ambarish Rapte
On Tue, 14 Nov 2023 12:47:51 GMT, Kevin Rushforth wrote: >> This PR updates the compiler on Windows to Visual Studio 2022 17.6.5 (from >> 17.5.0) to match JDK 22. >> >> I've run headless and headful tests. > > Kevin Rushforth has updated the pull request with a new target base due to a > merge

Re: RFR: 8318984: Update to Xcode 14.3.1 on macOS

2023-11-13 Thread Ambarish Rapte
On Mon, 13 Nov 2023 18:45:13 GMT, Kevin Rushforth wrote: > This PR updates the micro version of the compiler on macOS to 14.3.1 (from > 14.3) to match JDK 22. > > I've run headless and headful tests. Marked as reviewed by arapte (Reviewer). Marked as reviewed by arapte (Reviewer). --

Re: RFR: JDK-8284445: macOS 12 prints a warning when a function key shortcut is assigned to a menu [v3]

2023-11-07 Thread Ambarish Rapte
On Tue, 7 Nov 2023 16:04:56 GMT, Martin Fox wrote: >> @vlaaad >> Yes please, I think we should have a test along with this fix. A test is >> missing for this scenario. >> If a unit test is not feasible, can you please check to add a system test ? >> ( may be something like setting F1 as accele

Re: RFR: JDK-8284445: macOS 12 prints a warning when a function key shortcut is assigned to a menu [v3]

2023-11-07 Thread Ambarish Rapte
On Tue, 31 Oct 2023 08:05:56 GMT, vlaaad wrote: >> This PR addresses [JDK-8284445](https://bugs.openjdk.org/browse/JDK-8284445). >> >> Context: the warning might be more important than it seems. Our JavaFX-based >> desktop app has [an issue](https://github.com/defold/defold/issues/7845) — >> w

Re: RFR: JDK-8284445: macOS 12 prints a warning when a function key shortcut is assigned to a menu [v3]

2023-11-06 Thread Ambarish Rapte
On Mon, 6 Nov 2023 08:31:54 GMT, vlaaad wrote: >> When a function key is pressed >> [NSFunctionKeyMask](https://developer.apple.com/documentation/appkit/nsfunctionkeymask) >> is automatically included by the system. >> Hence it looks not breaking behavior with this change. >> As the warning is

Re: RFR: JDK-8284445: macOS 12 prints a warning when a function key shortcut is assigned to a menu [v3]

2023-11-02 Thread Ambarish Rapte
On Tue, 31 Oct 2023 08:05:56 GMT, vlaaad wrote: >> This PR addresses [JDK-8284445](https://bugs.openjdk.org/browse/JDK-8284445). >> >> Context: the warning might be more important than it seems. Our JavaFX-based >> desktop app has [an issue](https://github.com/defold/defold/issues/7845) — >> w

Re: RFR: 8319079: Missing range checks in decora [v3]

2023-11-02 Thread Ambarish Rapte
On Thu, 2 Nov 2023 06:55:29 GMT, Jayathirth D V wrote: >> In SW pipeline path of Box/Gaussian Blur/Shadow effects we are not checking >> for range when we read data from the source/destination buffers in native >> code. >> >> We need to add appropriate range checks in native JNI code also apar

Re: RFR: 8319066: Application window not always activated in macOS 14 Sonoma [v2]

2023-10-31 Thread Ambarish Rapte
On Mon, 30 Oct 2023 15:50:09 GMT, Jose Pereda wrote: >> On macOS 14. when a JavaFX application is launched from the command line it >> is not activated, and the terminal application remains active. >> >> This PR forces the application activation on macOS 14, in case the app >> didn't get activ

[jfx21u] Integrated: 8314484: Update Gradle to 8.4

2023-10-30 Thread Ambarish Rapte
On Mon, 30 Oct 2023 15:32:34 GMT, Ambarish Rapte wrote: > Clean backport to jfx21. > Verified a build locally on macOS. This pull request has now been integrated. Changeset: 7f704c24 Author:Ambarish Rapte URL: https://git.openjdk.org/jfx21u/

Re: [jfx21u] RFR: 8314484: Update Gradle to 8.4 [v2]

2023-10-30 Thread Ambarish Rapte
> Clean backport to jfx21. > Verified a build locally on macOS. Ambarish Rapte has updated the pull request incrementally with one additional commit since the last revision: Empty commit - Changes: - all: https://git.openjdk.org/jfx21u/pull/22/files - new:

[jfx21u] RFR: 8314484: Update Gradle to 8.4

2023-10-30 Thread Ambarish Rapte
Clean backport to jfx21. Verified a build locally on macOS. - Commit messages: - 8318115: Webkit build fails after gradle 8.4 - 8314484: Update Gradle to 8.4 - 8314486: JavaFX build uses deprecated features that will be removed in gradle 8 Changes: https://git.openjdk.org/jfx21u/

Integrated: 8318708: FX: Update copyright year in docs, readme files to 2024

2023-10-24 Thread Ambarish Rapte
On Tue, 24 Oct 2023 11:13:54 GMT, Ambarish Rapte wrote: > Update Copyright year in these 3 doc files to 2024. This pull request has now been integrated. Changeset: 2aa69e01 Author: Ambarish Rapte URL: https://git.openjdk.org/jfx/commit/2aa69e0120976c77df35adcdb4908df1b55f8281 St

Integrated: 8318714: Update copyright header for files modified in 2023

2023-10-24 Thread Ambarish Rapte
On Tue, 24 Oct 2023 12:28:34 GMT, Ambarish Rapte wrote: > Update copyright year to 2023 in files modified in this year 2023. This pull request has now been integrated. Changeset: bce15aa9 Author: Ambarish Rapte URL: https://git.openjdk.org/jfx/com

RFR: 8318714: Update copyright header for files modified in 2023

2023-10-24 Thread Ambarish Rapte
Update copyright year to 2023 in files modified in this year 2023. - Commit messages: - copyright year 2023 Changes: https://git.openjdk.org/jfx/pull/1268/files Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=1268&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8318714 Sta

RFR: 8318708: FX: Update copyright year in docs, readme files to 2024

2023-10-24 Thread Ambarish Rapte
Update Copyright year in these 3 doc files to 2024. - Commit messages: - update cpr-year in docs to 2024 Changes: https://git.openjdk.org/jfx/pull/1267/files Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=1267&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8318708 Stats:

Re: RFR: 8317508: Provide media support for libavcodec version 60

2023-10-18 Thread Ambarish Rapte
On Fri, 13 Oct 2023 01:24:36 GMT, Alexander Matveev wrote: > - Added support for libavcodec version 60. > - Tested on Ubuntu 23.10 with all possible media formats. > - Changed "--disable-yasm" to "--disable-asm". "--disable-asm" disables all > assembler optimization and "--disable-yasm" only x8

Integrated: 8318115: Webkit build fails after gradle 8.4

2023-10-14 Thread Ambarish Rapte
On Sat, 14 Oct 2023 05:17:44 GMT, Ambarish Rapte wrote: > Few dependency were missed to be added for webkit build. > With this change webkit build completes successfully. > Please review This pull request has now been integrated. Changeset: 6d2595a3 Author: Ambarish Rapte URL:

RFR: 8318115: Webkit build fails after gradle 8.4

2023-10-13 Thread Ambarish Rapte
Few dependency were missed to be added for webkit build. With this change webkit build completes successfully. Please review - Commit messages: - webkit build fail fix Changes: https://git.openjdk.org/jfx/pull/1261/files Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=1261&range=0

Integrated: 8314484: Update Gradle to 8.4

2023-10-12 Thread Ambarish Rapte
On Tue, 10 Oct 2023 17:39:53 GMT, Ambarish Rapte wrote: > Upgrade gradle to version 8.4. > It also includes a change for adding task dependency for windows platform > that was missed in [JDK-8314486](https://bugs.openjdk.org/browse/JDK-8314486) This pull request has now been i

RFR: 8314484: Update Gradle to 8.4

2023-10-10 Thread Ambarish Rapte
Upgrade gradle to version 8.4. It also includes a change for adding task dependency for windows platform that was missed in [JDK-8314486](https://bugs.openjdk.org/browse/JDK-8314486) - Commit messages: - chmod 644 gradlew - fix win task dependency - upgrade gradle to 8.4 Changes:

Integrated: 8314486: JavaFX build uses deprecated features that will be removed in gradle 8

2023-10-10 Thread Ambarish Rapte
On Mon, 9 Oct 2023 14:56:39 GMT, Ambarish Rapte wrote: > This is gradle only change, which fixes few warnings that are observed with > gradle 7.6 and which eventually result to an error with gradle 8.3 > With this change the warnings get fixed and no error is observed with grad

Re: RFR: 8314486: JavaFX build uses deprecated features that will be removed in gradle 8

2023-10-09 Thread Ambarish Rapte
On Mon, 9 Oct 2023 17:17:03 GMT, Kevin Rushforth wrote: > Looks like there some GHA failures which will need to be resolved. Possibly a > missing dependency? Yes, I had used `mustRunAfter` to specify order of execution of tasks but looks like it does not guarantee the order of execution. Chang

Re: RFR: 8314486: JavaFX build uses deprecated features that will be removed in gradle 8 [v2]

2023-10-09 Thread Ambarish Rapte
On Mon, 9 Oct 2023 22:48:18 GMT, Kevin Rushforth wrote: > I can see that you had to add a number of additional dependencies. I guess > these all represent latent dependency bugs in our `build.gradle`? Yes Kevin, Some tasks use output directory of other tasks without declaring dependency: This

Re: RFR: 8314486: JavaFX build uses deprecated features that will be removed in gradle 8 [v2]

2023-10-09 Thread Ambarish Rapte
. > > 2. Build with gradle 8.3 (using `--warning-mode all` gradle option) > 2.1 Without this change build would fail citing above warnings as errors > 2.2 With this change build completes successfully and above warnings do not > occur Ambarish Rapte has updated the pull request in

RFR: 8314486: JavaFX build uses deprecated features that will be removed in gradle 8

2023-10-09 Thread Ambarish Rapte
This is gradle only change, which fixes few warnings that are observed with gradle 7.6 and which eventually result to an error with gradle 8.3 With this change the warnings get fixed and no error is observed with gradle 8.3 Verified that several gradle tasks(all, sdk, javadoc, apps, shims, test)

Re: RFR: 8314486: JavaFX build uses deprecated features that will be removed in gradle 8

2023-10-09 Thread Ambarish Rapte
On Mon, 9 Oct 2023 14:56:39 GMT, Ambarish Rapte wrote: > This is gradle only change, which fixes few warnings that are observed with > gradle 7.6 and which eventually result to an error with gradle 8.3 > With this change the warnings get fixed and no error is observed with grad

Re: [jfx-tests] RFR: 8316097: Some Scenegraph/richtext tests fail due to IllegalStateException [v2]

2023-09-15 Thread Ambarish Rapte
On Fri, 15 Sep 2023 10:57:26 GMT, Ajit Ghaisas wrote: >> test/scenegraph/richtext/RichTextLabeledsTest.java >> test/scenegraph/richtext/RichTextMixedTest.java >> test/scenegraph/richtext/RichTextRectangleTest.java >> test/scenegraph/richtext/RichTextTextTest.java >> >> These tests were failing w

Re: [jfx-tests] RFR: 8316097: Some Scenegraph/richtext tests fail due to IllegalStateException

2023-09-15 Thread Ambarish Rapte
On Tue, 12 Sep 2023 10:33:07 GMT, Ajit Ghaisas wrote: > test/scenegraph/richtext/RichTextLabeledsTest.java > test/scenegraph/richtext/RichTextMixedTest.java > test/scenegraph/richtext/RichTextRectangleTest.java > test/scenegraph/richtext/RichTextTextTest.java > > These tests were failing with -

Re: [jfx-tests] RFR: 8315845: Exclude Scenegraph and Charts test classes that serve as a base class

2023-09-14 Thread Ambarish Rapte
On Thu, 7 Sep 2023 09:26:59 GMT, Ajit Ghaisas wrote: > A few SceneGraphTests and ControlsTests/chart test classes are abstract > classes and serve as base classes for other tests. > They are excluded from test execution and hence result in avoiding false > failure reports. Marked as reviewed b

Re: [jfx-tests] RFR: 8315928: Few Scenegraph and Charts tests fail due to resource not found error

2023-09-13 Thread Ambarish Rapte
On Fri, 8 Sep 2023 11:21:14 GMT, Ajit Ghaisas wrote: > Below tests fail due to "resource not found" error. > - test/scenegraph/binding/effects/IdentityTest.java > - test/scenegraph/lcd/controls/tests/AccordionTest.java > - Almost all ControlsTests/Chart tests > > Fix : > Moved required resources

Re: RFR: 8315958: Missing range checks in GlassPasteboard

2023-09-12 Thread Ambarish Rapte
On Mon, 11 Sep 2023 15:55:44 GMT, Kevin Rushforth wrote: > This PR adds missing range checks in the native `_getItemAsRawImage` and > `ByteArrayFromPixels` methods in `GlassPasteboard.m` (for both mac and ios). > > Note that these checks are _very_ unlikely to ever detect an error in > practic

<    1   2   3   4   5   >