[9] Review request for 8164096: ListChangeListener on ReadOnlyListWrapper's getReadOnlyProperty() does not reset change

2016-08-17 Thread Vadim Pakhnushev
Hi Chien, Could you please review the fix: https://bugs.openjdk.java.net/browse/JDK-8164096 http://cr.openjdk.java.net/~vadim/8164096/webrev.00/ Thanks, Vadim

Re: Exception running FXMark with JavaFX 9

2016-08-17 Thread Felix Bembrick
OK, I have read the article and followed Rahman's advice (thanks!) and here are my VM args now: -Djavafx.animation.fullspeed=true -addmods ALL-DEFAULT,java.se.ee -XaddExports:javafx.graphics/com.sun.javafx.stage=ALL-UNNAMED -XaddExports:javafx.base/com.sun.javafx.collections=ALL-UNNAMED -XaddExpor

Re: Exception running FXMark with JavaFX 9

2016-08-17 Thread Vadim Pakhnushev
Felix, The com.sun.javafx.perf.PerformanceTracker is in the javafx.graphics module. Vadim On 17.08.2016 10:50, Felix Bembrick wrote: OK, I have read the article and followed Rahman's advice (thanks!) and here are my VM args now: -Djavafx.animation.fullspeed=true -addmods ALL-DEFAULT,java.se.e

Re: Exception running FXMark with JavaFX 9

2016-08-17 Thread Felix Bembrick
Ah, thanks Vadim. That would certainly explain the exception! Cheers, Felix > On 17 Aug 2016, at 22:45, Vadim Pakhnushev > wrote: > > Felix, > > The com.sun.javafx.perf.PerformanceTracker is in the javafx.graphics module. > > Vadim > >> On 17.08.2016 10:50, Felix Bembrick wrote: >> OK, I

[PATCH] 8143596: FXCanvas does not forward touch gestures to embedded scene

2016-08-17 Thread Alexander Nyssen
Hi all, Kevin, attached please find a patch for JDK-8143596 (FXCanvas does not forward touch gestures to embedded scene). The patch depends on the most recent patch I provided for JDK-8161282 (revision 15_08_16, which was already uploaded). I

End-to-end performance tests for distributed JavaFX application

2016-08-17 Thread Daniel Glöckner
Hi, We're looking at developing a performance benchmark suite for our distributed JavaFX application. We want end-to-end metrics which include rendering / event processing in the UI. Example 1) @T_0 server sends update to client, @T_1 client receives the update, @T_2 the update is rendered. Mea

Re: [PATCH] 8161282: FXCanvas does not forward horizontal mouse scroll events to the embedded scene

2016-08-17 Thread Kevin Rushforth
Uploaded the new version. Alexander Zvegintsev will sponsor this patch, and I will review it. My one quick comment -- in addition to the wild-card imports which Alexander Z caught, and said he would take care of -- is that the format of the commit message is not quite right. See the following

Re: [PATCH] 8143596: FXCanvas does not forward touch gestures to embedded scene

2016-08-17 Thread Kevin Rushforth
I uploaded the patch. I also added a comment [1] to the bug report with a concern about updating the dependent SWT library. You can read the details in the bug report. -- Kevin [1] https://bugs.openjdk.java.net/browse/JDK-8143596?focusedCommentId=13989256&page=com.atlassian.jira.plugin.system

[9] Review request for JDK-8160073: menu issues in case of two-display configuration

2016-08-17 Thread Jim Graham
JBS: https://bugs.openjdk.java.net/browse/JDK-8160073 webrev: http://cr.openjdk.java.net/~flar/JDK-8160073/webrev.00/ The details are in the comments in the bug report. To summarize: - send a notification when DPI changes to reevaluate scene.getXY() - correctly scale event screen coordinates fo