Re: Re: Seeking help with latest master ...

2020-04-03 Thread Eric Bresie
Are you behind a proxy? I had problems with another application which required registering certificates and related credentials keys to be added for java. Eric Bresie ebre...@gmail.com > On April 2, 2020 at 2:20:44 PM CDT, Johan Vos wrote: > I have no scientific evidence for this, but I remember

Re: [Rev 43] RFR: 8236651: Simplify and update glass gtk backend

2020-04-03 Thread Thiago Milczarek Sayao
> This proposed change does the following: > > - Ports DND target to use GTK reducing code size and adding extra text / > image formats (such as .gif); > - Use gtk signals instead of gdk events (also to reduce code size); > - Simplifies geometry (sizing/positioning) with a more straightforward co

Re: Stuck attempting to run/develop a manual unit test

2020-04-03 Thread Craig Cavanaugh
Jeanette, Thank you! Much appreciated and a much better approach than a manual test. Well on my way now! Thanks, Craig On Fri, Apr 3, 2020 at 7:48 AM Jeanette Winzenburg wrote: > > Hi Craig, > > no manual test needed - you can implement a normal (unit) test with > the help or VirtualFlowTestU

Re: RFR: 8236651: Simplify and update glass gtk backend

2020-04-03 Thread Thiago Milczarek Sayao
On Fri, 3 Apr 2020 15:21:29 GMT, Kevin Rushforth wrote: >> I will keep testing it, but I think it's looking good. > > I see a lot of work going into this. > > In order for this to progress beyond the prototype or concept phase, we will > need to have a discussion on the > openjfx-dev mailing li

Re: Stuck attempting to run/develop a manual unit test

2020-04-03 Thread Craig Cavanaugh
Thanks! That was the trick! On Fri, Apr 3, 2020 at 6:52 AM Ajit Ghaisas wrote: > Hi Craig, > > Here is how I run the ButtonMnemonicPositionTest. > > $ export FX_LIB=/Users/AG/aaa/jfx/build/sdk/lib > $ cd /Users/AG/aaa/jfx/tests/manual/UI > $ javac --module-path $FX_LIB --add-mod

Re: Alternatives for JDK-8185886: Improve scrolling performance of TableView and TreeTableView

2020-04-03 Thread Kevin Rushforth
That's a fair question. Hard to say, but my guess is that if we do the first, the incremental improvements of the second might not be worth the effort or risk. And if we go with the second, we would need another test case that hits the slow iteration problem when removing a listener from a lar

Re: Alternatives for JDK-8185886: Improve scrolling performance of TableView and TreeTableView

2020-04-03 Thread Scott Palmer
Assuming testing and performance/memory analysis leads to the conclusion that the risks are worth it, would it make sense to do both? Would we get a greater benefit from the combined effects? Or is the incremental improvement of including the second fix (whichever it may be) no longer significan

Alternatives for JDK-8185886: Improve scrolling performance of TableView and TreeTableView

2020-04-03 Thread Kevin Rushforth
We now have two pull requests under review that propose to solve the poor scrolling performance of TableView and TreeTableView, as tracked by JDK-8185886 [1] The first one, PR #108 [2], proposes a change in the bindings ExpressionHelper code relating to the cleaning up of listeners (changing

Re: Question ad CSR (Re: A WIP for JDK-8238080 for review/discussion (Re: "Internal review ID 9063426: "FXMLLoader: if script engines implement javax.script.Compilabel compile scripts"

2020-04-03 Thread Kevin Rushforth
Regarding the CSR, that's usually done after the enhancement is agreed upon in principle, and the review is far enough along that you are ready to write up the spec changes, new API, and/or new interfaces. Since you don't have direct JBS access you will need a sponsor to do that part for you,

Question ad CSR (Re: A WIP for JDK-8238080 for review/discussion (Re: "Internal review ID 9063426: "FXMLLoader: if script engines implement javax.script.Compilabel compile scripts"

2020-04-03 Thread Rony G. Flatscher
Hi Kevin, thank you for your feedback and actions! Ad "please create a CSR request and add link to it in JDK-8238080": preliminarily researching [CSR FAQ] (https://wiki.openjdk.java.net/display/csr/CSR+FAQs) the question "Q: How do I create a CSR ?" gets answered with: > "A: _Do not directly c

Re: RFR: 8236651: Simplify and update glass gtk backend

2020-04-03 Thread Kevin Rushforth
On Fri, 3 Apr 2020 02:11:12 GMT, Thiago Milczarek Sayao wrote: >> Test on 16.04 (without Webkit - it does not build on 16.04 anymore) >> >> Updated April 2nd. >> >> ![image](https://user-images.githubusercontent.com/30704286/78316727-c14db300-7536-11ea-86e9-4d5c56e4d92c.png) >> >> Note: DateP

Re: RFR: 8236971: [macos] Gestures handled incorrectly due to missing events

2020-04-03 Thread Michael Paus
On Thu, 2 Apr 2020 14:55:35 GMT, Kevin Rushforth wrote: > As noted in the JBS issue, this bug is a result of a deliberate change by > Apple that affects applications (in this case > the JDK) linked against MacOSX 10.11 SDK or later -- they no longer call two > methods that we are relying on, >

Re: RFR: 8236971: [macos] Gestures handled incorrectly due to missing events

2020-04-03 Thread Kevin Rushforth
On Fri, 3 Apr 2020 14:14:36 GMT, Michael Paus wrote: >> These are all separate issues that could be explored with follow-on bugs. >> I'm not sure what the right behavior is for >> these or whether there is enough information from the OS to do anything >> about them (e.g., I suspect there is no

Re: RFR: 8236971: [macos] Gestures handled incorrectly due to missing events

2020-04-03 Thread Michael Paus
On Fri, 3 Apr 2020 12:35:35 GMT, Kevin Rushforth wrote: >> I've built a fresh JFX, with your changes applied to master, this morning. I >> can confirm that the primary bug seems to >> be fixed with these changes but I have observed some behaviour where I am >> not sure whether it is correct or

Re: RFR: 8236971: [macos] Gestures handled incorrectly due to missing events

2020-04-03 Thread Kevin Rushforth
On Fri, 3 Apr 2020 11:32:12 GMT, Michael Paus wrote: >> @arapte Can you be one of the reviewers? >> >> @mipastgt I ran it against your test application, but if you have other >> tests you could run, that would be helpful. > > I've built a fresh JFX, with your changes applied to master, this mor

Re: RFR: 8129123: ComboBox popup list view does not scrollTo when ComboBox value/selection is set

2020-04-03 Thread Jeanette Winzenburg
On Fri, 3 Apr 2020 12:03:33 GMT, Jeanette Winzenburg wrote: >>> Can you please provide a unit test? One that fails before your fix and >>> passes after your fix. >> >> I can provide a manual test the next couple of days that demonstrates it >> before and after, but I'm not sure how to >> crea

Re: RFR: 8129123: ComboBox popup list view does not scrollTo when ComboBox value/selection is set

2020-04-03 Thread Jeanette Winzenburg
On Thu, 2 Apr 2020 23:28:19 GMT, Craig Cavanaugh wrote: >> Can you please provide a unit test? One that fails before your fix and >> passes after your fix. > >> Can you please provide a unit test? One that fails before your fix and >> passes after your fix. > > I can provide a manual test the

Re: Stuck attempting to run/develop a manual unit test

2020-04-03 Thread Jeanette Winzenburg
Hi Craig, no manual test needed - you can implement a normal (unit) test with the help or VirtualFlowTestUtils: grab the flow, get its first/last visible cell and check whether their range includes to cell we want to see, something like: @Test public void testScrollInSkin() {

Re: RFR: 8236971: [macos] Gestures handled incorrectly due to missing events

2020-04-03 Thread Michael Paus
On Thu, 2 Apr 2020 22:04:55 GMT, Kevin Rushforth wrote: >> As noted in the JBS issue, this bug is a result of a deliberate change by >> Apple that affects applications (in this case >> the JDK) linked against MacOSX 10.11 SDK or later -- they no longer call two >> methods that we are relying on

RFR: 8185886: Improve scrolling performance of TableView and TreeTableView

2020-04-03 Thread yosbits
If there are many columns, the current TableView will stall scrolling. Resolving this performance issue requires column virtualization. Virtualization mode is enabled when the row height is fixed by the following method. `tableView.setFixedCellSize(height)` This proposal includes a fix because

Re: RFR: 8185886: Improve scrolling performance of TableView and TreeTableView

2020-04-03 Thread Dalibor Topic
On Mon, 9 Mar 2020 17:00:06 GMT, Nir Lisker wrote: >> I took the liberty of pointing out some formatting errors > > The title of your PR should match the title of the JBS issue. Moreover, #108 > already tackles this issue with a > different approach and 2 PRs on the same issue can't be intetgrat

Re: RFR: 8185886: Improve scrolling performance of TableView and TreeTableView

2020-04-03 Thread Martin Polakovic
On Mon, 24 Feb 2020 07:39:43 GMT, yosbits wrote: > If there are many columns, the current TableView will stall scrolling. > Resolving this performance issue requires column > virtualization. Virtualization mode is enabled when the row height is fixed > by the following method. > `tableView.set

Re: RFR: 8197991: Selecting many items in a TableView is very slow

2020-04-03 Thread Dalibor Topic
On Wed, 26 Feb 2020 07:37:06 GMT, yosbits wrote: > https://bugs.openjdk.java.net/browse/JDK-8197991 > > The performance of the selectAll and selectRange methods of the > MultiSelectionModel class has been greatly improved. > > This greatly improves the response when selecting multiple items i

Re: RFR: 8185886: Improve scrolling performance of TableView and TreeTableView

2020-04-03 Thread yosbits
On Thu, 2 Apr 2020 17:38:00 GMT, Dalibor Topic wrote: >> The title of your PR should match the title of the JBS issue. Moreover, #108 >> already tackles this issue with a >> different approach and 2 PRs on the same issue can't be intetgrated. Since >> I'm not familiar with this code, I can't >>

Re: RFR: 8185886: Improve scrolling performance of TableView and TreeTableView

2020-04-03 Thread Nir Lisker
On Thu, 27 Feb 2020 21:29:27 GMT, Martin Polakovic wrote: >> If there are many columns, the current TableView will stall scrolling. >> Resolving this performance issue requires column >> virtualization. Virtualization mode is enabled when the row height is fixed >> by the following method. >>

Re: RFR: 8185886: Improve scrolling performance of TableView and TreeTableView

2020-04-03 Thread yosbits
On Mon, 24 Feb 2020 07:39:43 GMT, yosbits wrote: > If there are many columns, the current TableView will stall scrolling. > Resolving this performance issue requires column > virtualization. Virtualization mode is enabled when the row height is fixed > by the following method. > `tableView.set

Re: Stuck attempting to run/develop a manual unit test

2020-04-03 Thread Ajit Ghaisas
Hi Craig, Here is how I run the ButtonMnemonicPositionTest. $ export FX_LIB=/Users/AG/aaa/jfx/build/sdk/lib $ cd /Users/AG/aaa/jfx/tests/manual/UI $ javac --module-path $FX_LIB --add-modules javafx.controls ButtonMnemonicPositionTest.java $ java --module-path $FX_LIB --add-mo

RFR: 8197991: Selecting many items in a TableView is very slow

2020-04-03 Thread yosbits
https://bugs.openjdk.java.net/browse/JDK-8197991 The performance of the selectAll and selectRange methods of the MultiSelectionModel class has been greatly improved. This greatly improves the response when selecting multiple items in ListView and TableView. However, in TreeView / TreeTableView

Stuck attempting to run/develop a manual unit test

2020-04-03 Thread Craig Cavanaugh
I am trying to create a manual unit test for RFR: 8129123 and I'm using the existing ButtonMnemonicPositionTest.java as a template to make sure everything is working correctly. I'm using an Ubuntu 18.04.4 build environment using the default 11.0.6 openjdk. openjfx is successfully compiling and pas

Re: [Rev 02] RFR: 8241455: Memory leak on replacing selection/focusModel

2020-04-03 Thread Ajit Ghaisas
On Thu, 2 Apr 2020 09:11:57 GMT, Jeanette Winzenburg wrote: >> Several controls with selection/focusModels leave memory leaks on replacing >> the model. >> >> Added tests for all such, those for the misbehaving models fail before and >> pass after the fix. >> >> for convenience, the bug refe