Integrated: 8159048: Animation and AnimationTimer methods must be called on JavaFX Application thread

2023-08-14 Thread Jose Pereda
On Mon, 3 Jul 2023 15:49:31 GMT, Jose Pereda wrote: > This PR adds a check to the Animation and AnimationTimer public methods to > verify that these are called from the JavaFX Application thread. If the call > is done from any other thread, an IllegalStateException will be thrown. > > This wil

Re: RFR: 8309558: Create implementation of NSAccessibilityCheckBox protocol [v3]

2023-08-14 Thread Alexander Zuev
On Mon, 14 Aug 2023 18:38:39 GMT, Alexander Zuev wrote: >> also >> 8309629: Create implementation of NSAccessibilityRadioButton protocol >> >> Create implementation of NSAccessibilityCheckBox and >> NSAccessibilityRadioButton protocols >> Add workaround for the wrong focus owner announcement wi

Re: RFR: 8309558: Create implementation of NSAccessibilityCheckBox protocol [v2]

2023-08-14 Thread Alexander Zuev
On Mon, 7 Aug 2023 14:10:09 GMT, Kevin Rushforth wrote: >> modules/javafx.graphics/src/main/native-glass/mac/a11y/AccessibleBase.m line >> 174: >> >>> 172: jobject jresult = NULL; >>> 173: GET_MAIN_JENV; >>> 174: if (env == NULL) return NULL; >> >> This generates a warning: >> `war

Re: RFR: JDK-8313628: Column drag header, overlay and line are not correctly aligned

2023-08-14 Thread Andy Goryachev
On Fri, 4 Aug 2023 07:36:08 GMT, Marius Hanl wrote: >> modules/javafx.controls/src/test/java/test/javafx/scene/control/skin/TableViewSkinTest.java >> line 79: >> >>> 77: @Test >>> 78: void testInitialColumnResizeNodePositions() { >>> 79: TableView tableView = new TableView<>();

Re: RFR: 8309558: Create implementation of NSAccessibilityCheckBox protocol [v3]

2023-08-14 Thread Alexander Zuev
> also > 8309629: Create implementation of NSAccessibilityRadioButton protocol > > Create implementation of NSAccessibilityCheckBox and > NSAccessibilityRadioButton protocols > Add workaround for the wrong focus owner announcement with radio buttons > Add workaround for wrong magnifier text on bu

Re: RFR: 8313900: Possible NULL pointer access in NativeAudioSpectrum and NativeVideoBuffer

2023-08-14 Thread Kevin Rushforth
On Mon, 14 Aug 2023 16:46:33 GMT, Andy Goryachev wrote: >> - Fixed by checking for `NULL` pointer after memory allocation. >> - In `NativeVideoBuffer` `std::nothrow` was added when allocating `jint` >> array, so `new` will return `NULL` instead of throwing exception. This done >> for consistenc

Re: RFR: 8313900: Possible NULL pointer access in NativeAudioSpectrum and NativeVideoBuffer

2023-08-14 Thread Andy Goryachev
On Mon, 7 Aug 2023 23:33:37 GMT, Alexander Matveev wrote: > - Fixed by checking for `NULL` pointer after memory allocation. > - In `NativeVideoBuffer` `std::nothrow` was added when allocating `jint` > array, so `new` will return `NULL` instead of throwing exception. This done > for consistency

RFR: 8087700: [KeyCombination, Mac] One key event sent on Mac versus two on Windows

2023-08-14 Thread Martin Fox
A KeyCharacterCombination should match a key if the target character is printed on that key. For example, the user should be able to invoke the `Shortcut+'+' ` combination by holding down the Shortcut key and pressing a key that has '+' printed on it. This should work even if '+' is a shifted sy

Re: RFR: 8313321 : Set minimum python version in WebKit cmake scripts

2023-08-14 Thread Hima Bindu Meda
On Thu, 10 Aug 2023 23:00:11 GMT, Kevin Rushforth wrote: > I tested this on my MacBook Pro running macOS 13.3.1 and discovered that it > has python 3.7.x: > > ``` > Could NOT find Python3: Found unsuitable version "3.7.3", but required is > at least "3.8.0" (found /usr/bin/python3, found co

Re: RFR: 8313321 : Set minimum python version in WebKit cmake scripts [v2]

2023-08-14 Thread Hima Bindu Meda
> Minimum version of python ,to run webkit build , on linux is set to 3.6 and > for other platforms, minimum version of python is set to 3.8. > > Verified build on all platforms. No issue seen Hima Bindu Meda has updated the pull request incrementally with one additional commit since the last r

Re: RFR: JDK-8199216: Quadratic layout time with nested nodes and pseudo-class in style sheet [v3]

2023-08-14 Thread John Hendrikx
On Mon, 14 Aug 2023 13:16:06 GMT, Johan Vos wrote: > > Yes, it would do. In this case, I presume that this PR is dependent on PR > > #1070, which will have a CSR. Once that is integrated, and the updated > > master is merged into this PR branch, I expect that all changes to public > > API in t

Re: RFR: 8159048: Animation and AnimationTimer methods must be called on JavaFX Application thread [v2]

2023-08-14 Thread Jose Pereda
On Mon, 31 Jul 2023 17:47:06 GMT, Jose Pereda wrote: >> This PR adds a check to the Animation and AnimationTimer public methods to >> verify that these are called from the JavaFX Application thread. If the call >> is done from any other thread, an IllegalStateException will be thrown. >> >> Th

Re: RFR: 8159048: Null PulseReceiver in AbstractMasterTimer [v2]

2023-08-14 Thread Kevin Rushforth
On Mon, 14 Aug 2023 15:04:28 GMT, Jose Pereda wrote: > Makes sense, I've updated the title of both JBS issue and CSR. @jperedadnr You also need to update this PR title. Also, please "Finalize" the CSR when ready, since that will trigger the final review of the CSR. - PR Comment: h

Re: RFR: 8159048: Null PulseReceiver in AbstractMasterTimer [v2]

2023-08-14 Thread Jose Pereda
On Mon, 31 Jul 2023 17:47:06 GMT, Jose Pereda wrote: >> This PR adds a check to the Animation and AnimationTimer public methods to >> verify that these are called from the JavaFX Application thread. If the call >> is done from any other thread, an IllegalStateException will be thrown. >> >> Th

Re: RFR: JDK-8312058: Documentation improvements for subscription based listeners [v6]

2023-08-14 Thread Andy Goryachev
On Fri, 11 Aug 2023 22:03:28 GMT, John Hendrikx wrote: >> Incorporates documentation review comments from #1069. >> >> This PR should be kept minimal so it can be backported to JFX21; it should >> only contain documentation changes. > > John Hendrikx has updated the pull request incrementally w

Re: RFR: 8159048: Null PulseReceiver in AbstractMasterTimer

2023-08-14 Thread Kevin Rushforth
On Mon, 31 Jul 2023 17:39:29 GMT, Jose Pereda wrote: >> This PR adds a check to the Animation and AnimationTimer public methods to >> verify that these are called from the JavaFX Application thread. If the call >> is done from any other thread, an IllegalStateException will be thrown. >> >> Th

Re: Center text in a table once a button has been added dynamically

2023-08-14 Thread Davide Perini
I answer my self for posters... .table-view .table-cell { -fx-alignment:center-left; } done the trick. thanks Davide Il 14/08/2023 15:34, Davide Perini ha scritto: Hi all, hope that someone can help with this :) I have a TableView like this: If I add a remove button in the table like

Center text in a table once a button has been added dynamically

2023-08-14 Thread Davide Perini
Hi all, hope that someone can help with this :) I have a TableView like this: If I add a remove button in the table like this: private TableColumngetSatelliteVoidTableColumn() { TableColumn colBtn =new TableColumn<>(""); colBtn.setMaxWidth(Constants.REMOVE_BTN_TABLE); Callback, Ta

Re: RFR: JDK-8199216: Quadratic layout time with nested nodes and pseudo-class in style sheet [v8]

2023-08-14 Thread Johan Vos
On Fri, 9 Jun 2023 12:45:02 GMT, John Hendrikx wrote: >> This fix introduces immutable sets of `PseudoClass` almost everywhere, as >> they are rarely modified. These are re-used by caching them in a new class >> `ImmutablePseudoClassSetsCache`. >> >> In order to make this work, `BitSet` had t

Re: RFR: JDK-8199216: Quadratic layout time with nested nodes and pseudo-class in style sheet [v3]

2023-08-14 Thread Johan Vos
On Tue, 25 Apr 2023 05:36:32 GMT, John Hendrikx wrote: > > Match is a public class, does this PR need a CSR? > > Yes, it would do. In this case, I presume that this PR is dependent on PR > #1070, which will have a CSR. Once that is integrated, and the updated master > is merged into this PR br

Re: RFR: 8314149: Clipboard does inexact string comparison on mime type [v2]

2023-08-14 Thread Ambarish Rapte
On Fri, 11 Aug 2023 12:10:01 GMT, Kevin Rushforth wrote: > The code change looks good. However, the new test passes for me either with > or without your fix. Hi Kevin, I re-looked into the test. The test seems to be insufficient to test the change. That test and other test in the file Clipboar

Re: RFR: 8314149: Clipboard does inexact string comparison on mime type [v2]

2023-08-14 Thread Ambarish Rapte
> The mime name should be exactly same as the mime type supported by platform. > Hence the comparison should not match partially similar names. > Added an invalid unit test. > > @kevinrushforth @jayathirthrao please review Ambarish Rapte has updated the pull request incrementally with one additio