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

2024-02-27 Thread Johan Vos
On Mon, 26 Feb 2024 15:10:33 GMT, Kevin Rushforth wrote: >> Johan Vos has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Add additional test for IOOBE detection. >> This test comes from JDK-8323787 > > Getting back to this I ran the t

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

2024-02-27 Thread Johan Vos
> 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 with a new target base due to a merge or a rebase. The incremental webrev excludes the un

Re: RFR: 8319844 : Text/TextFlow.hitTest() is incorrect in RTL orientation [v13]

2024-02-27 Thread Karthik P K
On Mon, 26 Feb 2024 11:35:06 GMT, Karthik P K wrote: >> tests/system/src/test/java/test/robot/javafx/scene/RTLTextFlowCharacterIndexTest.java >> line 238: >> >>> 236: >>> 237: @Test >>> 238: public void testTextAndTextFlowHitInfoForRTLEnglishText() throws >>> Exception { >> >> I just

Re: RFR: 8319844 : Text/TextFlow.hitTest() is incorrect in RTL orientation [v14]

2024-02-27 Thread Karthik P K
> In the `getHitInfo()` method of PrismTextLayout, RTL node orientation > conditions were not considered, hence hit test values such as character index > and insertion index values were incorrect. > > Added checks for RTL orientation of nodes and fixed the issue in > `getHitInfo()` to calculat

Re: RFR: 8319844 : Text/TextFlow.hitTest() is incorrect in RTL orientation [v13]

2024-02-27 Thread Karthik P K
On Tue, 27 Feb 2024 05:09:20 GMT, Karthik P K wrote: >> yes, this bothered me from the start. I did have a test case in the MT with >> two text nodes with the same text, and it seemed to work correctly. or did >> I miss something? > > Actually most of the repeating text cases are handled. I s

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

Re: RFR: 8319844 : Text/TextFlow.hitTest() is incorrect in RTL orientation [v14]

2024-02-27 Thread John Hendrikx
On Tue, 27 Feb 2024 13:51:08 GMT, Karthik P K wrote: >> In the `getHitInfo()` method of PrismTextLayout, RTL node orientation >> conditions were not considered, hence hit test values such as character >> index and insertion index values were incorrect. >> >> Added checks for RTL orientation of

RFR: 8326618 : Replace usage of deprecated method getId() in Thread

2024-02-27 Thread Anirvan Sarkar
Replace Thread.currentThread().getId() with Thread.currentThread().threadId() - Commit messages: - Replace Thread.currentThread().getId() with Thread.currentThread().threadId() Changes: https://git.openjdk.org/jfx/pull/1383/files Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=138

Re: RFR: 8326618 : Replace usage of deprecated method getId() in Thread

2024-02-27 Thread John Hendrikx
On Tue, 27 Feb 2024 17:58:13 GMT, Anirvan Sarkar wrote: > Replace Thread.currentThread().getId() with Thread.currentThread().threadId() Looks okay. `threadId` was added in JDK 19, but since we're on 21 now that should be fine. - Marked as reviewed by jhendrikx (Committer). PR Re

Re: RFR: JDK-8285893: Hiding dialog and showing new one causes dialog to be frozen [v2]

2024-02-27 Thread Martin Fox
On Thu, 22 Feb 2024 23:13:26 GMT, Marius Hanl wrote: >> This PR fixes the dialog freeze problem once and for all. >> >> This one is a bit tricky to understand, here is how it works: >> This bug happens on every platform, although the implementation of nested >> event loops differs on every pla

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

2024-02-27 Thread Jayathirth D V
> 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. > > Also while updating to 9f noticed that many files don't ha