Re: RFR: 8089398: [ChoiceBox, ComboBox] throws NPE on setting value on null selectionModel [v6]

2021-09-08 Thread Ajit Ghaisas
On Tue, 27 Jul 2021 12:41:00 GMT, Marius Hanl wrote: >> This PR fixes multiple NPEs in Choice-and ComboBox, when the selection model >> is null. >> >> ChoiceBox: >> - Null check in **valueProperty()** listener >> >> ComboBox: >> - Null check in **editableProperty()** listener >> - Null check

Re: RFR: 8090547: Allow for transparent backgrounds in WebView [v7]

2021-09-08 Thread Kevin Rushforth
On Wed, 8 Sep 2021 21:10:27 GMT, Kevin Rushforth wrote: > I looked at the generated javadoc and this will read better without a > paragraph break. When you remove the paragraph break, please also remove the blank line to make it clearer to future readers of the code that it is intended to be p

Re: RFR: 8090547: Allow for transparent backgrounds in WebView [v7]

2021-09-08 Thread Kevin Rushforth
On Wed, 8 Sep 2021 16:15:36 GMT, Jose Pereda wrote: >> Currently, `WebPage` has already a public `setBackgroundColor()` method, but >> the class is not public. Therefore, public API is needed in `WebView` to >> allow developers access to it. >> >> In line with the `fontSmoothingType` property,

Re: RFR: 8090547: Allow for transparent backgrounds in WebView [v7]

2021-09-08 Thread Jose Pereda
> Currently, `WebPage` has already a public `setBackgroundColor()` method, but > the class is not public. Therefore, public API is needed in `WebView` to > allow developers access to it. > > In line with the `fontSmoothingType` property, this PR provides public > support for setting the backgro

Integrated: 8273324: IllegalArgumentException: fromIndex(0) > toIndex(-1) after clear and select TableCell

2021-09-08 Thread Michael Strauß
On Fri, 3 Sep 2021 18:19:45 GMT, Michael Strauß wrote: > This PR fixes the exception thrown by the sample code in > [8273324](https://bugs.openjdk.java.net/browse/JDK-8273324), while retaining > the incorrect behavior in the scenario described. This pull request has now been integrated. Chang

Re: RFR: 8273324: IllegalArgumentException: fromIndex(0) > toIndex(-1) after clear and select TableCell [v3]

2021-09-08 Thread Jose Pereda
On Tue, 7 Sep 2021 10:40:01 GMT, Michael Strauß wrote: >> This PR fixes the exception thrown by the sample code in >> [8273324](https://bugs.openjdk.java.net/browse/JDK-8273324), while retaining >> the incorrect behavior in the scenario described. > > Michael Strauß has updated the pull request

Re: RFR: 8089398: [ChoiceBox, ComboBox] throws NPE on setting value on null selectionModel [v6]

2021-09-08 Thread Ajit Ghaisas
On Wed, 8 Sep 2021 12:20:57 GMT, Kevin Rushforth wrote: > > The latest patch does not apply cleanly due to the change in copyright > > headers. > > @aghaisas If you fetch the PR branch and merge master (which is what I > usually do when testing a PR), git is able to auto-resolve it; this is wh

Re: RFR: 8273324: IllegalArgumentException: fromIndex(0) > toIndex(-1) after clear and select TableCell [v3]

2021-09-08 Thread Kevin Rushforth
On Tue, 7 Sep 2021 10:40:01 GMT, Michael Strauß wrote: >> This PR fixes the exception thrown by the sample code in >> [8273324](https://bugs.openjdk.java.net/browse/JDK-8273324), while retaining >> the incorrect behavior in the scenario described. > > Michael Strauß has updated the pull request

Re: RFR: 8090547: Allow for transparent backgrounds in WebView [v6]

2021-09-08 Thread Kevin Rushforth
On Mon, 6 Sep 2021 09:28:20 GMT, Jose Pereda wrote: >> modules/javafx.web/src/main/java/com/sun/webkit/WebPage.java line 97: >> >>> 95: >>> 96: private int fontSmoothingType; >>> 97: private Color backgroundColor = Color.WHITE; >> >> This might be a problem, since there are code paths

Re: RFR: 8090547: Allow for transparent backgrounds in WebView [v5]

2021-09-08 Thread Kevin Rushforth
On Sun, 29 Aug 2021 06:36:40 GMT, Michel Jung wrote: >> Jose Pereda has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Color to int32 conversion and more changes based on feedback > > modules/javafx.web/src/main/java/com/sun/webkit/WebPage

Re: RFR: 8090547: Allow for transparent backgrounds in WebView [v4]

2021-09-08 Thread Kevin Rushforth
On Mon, 6 Sep 2021 11:05:42 GMT, Jose Pereda wrote: >> I don't doubt that the area should be cleared. What I was questioning is >> whether this was the best place to do it? I'd be OK with it if you document >> it. > > The two options I see so far to clear the area are the one I've committed >

Re: RFR: 8089398: [ChoiceBox, ComboBox] throws NPE on setting value on null selectionModel [v6]

2021-09-08 Thread Kevin Rushforth
On Wed, 8 Sep 2021 06:32:06 GMT, Ajit Ghaisas wrote: > The latest patch does not apply cleanly due to the change in copyright > headers. @aghaisas If you fetch the PR branch and merge master (which is what I usually do when testing a PR), git is able to auto-resolve it; this is why Skara didn'

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac

2021-09-08 Thread Kevin Rushforth
On Wed, 8 Sep 2021 10:37:40 GMT, Florian Kirmaier wrote: > When using Swing it's possible to generate a Deadlock. > It's related to the nested eventloop started in enterFullScreenExitingLoop - > and the RenderLock aquired when using setView in Scene. > Sample Programm and Threaddump are added

Integrated: 8273138: BidirectionalBinding fails to observe changes of invalid properties

2021-09-08 Thread Michael Strauß
On Sun, 29 Aug 2021 04:12:22 GMT, Michael Strauß wrote: > This PR fixes a bug that was introduced in #454. > > Since this fix might impact the performance considerations in the original > PR, I ran a performance benchmark against the previous `ChangeListener`-based > implementation, which stil

RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac

2021-09-08 Thread Florian Kirmaier
When using Swing it's possible to generate a Deadlock. It's related to the nested eventloop started in enterFullScreenExitingLoop - and the RenderLock aquired when using setView in Scene. Sample Programm and Threaddump are added to the ticket. Removing the nested loop fixes the Problem. I hope

Re: RFR: 8273138: BidirectionalBinding fails to observe changes of invalid properties

2021-09-08 Thread Ambarish Rapte
On Sun, 29 Aug 2021 04:12:22 GMT, Michael Strauß wrote: > This PR fixes a bug that was introduced in #454. > > Since this fix might impact the performance considerations in the original > PR, I ran a performance benchmark against the previous `ChangeListener`-based > implementation, which stil

Re: RFR: 8271395: Fixing crash at printing [v2]

2021-09-08 Thread Florian Kirmaier
> This PR switches the Thread to the QuantumRenderer, in the case the Disposer > is called from another Thread - the printing Thread. > I'm open for better solutions on how to fix this Issue. > Initially i thought there is also a Race Condition in the resource pool, but > a new one is created for

Re: RFR: 8271395: Fixing crash at printing [v2]

2021-09-08 Thread Florian Kirmaier
On Mon, 6 Sep 2021 13:33:11 GMT, Kevin Rushforth wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> JDK-8271395 >> QuantumRenderer is no longer public > > modules/javafx.graphics/src/main/java/com/sun/javafx/tk/