On Thu, 17 Oct 2019 15:28:31 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
> On Thu, 17 Oct 2019 15:28:30 GMT, mruzicka > <github.com+562920+mruzi...@openjdk.org> wrote: > >> On Thu, 17 Oct 2019 15:28:28 GMT, mruzicka >> <github.com+562920+mruzi...@openjdk.org> wrote: >> >>> Secondary event loop introduced as a means of synchronization with the >>> JavaFX Application thread in [1] never terminates as the >>> SecondaryLoop.exit() call is not reached because the thread is blocked in >>> the SecondaryLoop.enter() call. >>> This patch fixes the problem by submitting the UI work (including the call >>> to the SecondaryLoop.exit() method) before entering the secondary loop. >>> >>> [1] >>> https://github.com/openjdk/jfx/commit/7cf2dfa0b3c5bfd0f1a2de36d46b62f7e9e256c4 >>> >>> ---------------- >>> >>> Commits: >>> - 9483ccde: 8231372: Correctly terminate secondary event loop in >>> JFXPanel.setScene() >>> >>> Changes: https://git.openjdk.java.net/jfx/pull/16/files >>> Webrev: https://webrevs.openjdk.java.net/jfx/16/webrev.00 >>> Issue: https://bugs.openjdk.java.net/browse/JDK-8231372 >>> Stats: 6 lines in 1 file changed: 1 ins; 2 del; 3 mod >>> Patch: https://git.openjdk.java.net/jfx/pull/16.diff >>> Fetch: git fetch https://git.openjdk.java.net/jfx pull/16/head:pull/16 >> >> @kevinrushforth I believe you should be able to move forward with the review >> of this PR now as I've filed a bug report for the problem and signed the OCA >> as required. (As I did not know I needed to have my github account >> associated with the OCA, I did not include it in my details. I've asked >> Dalibor Topic from the Java SE PM team to help me have that fixed or get in >> touch with you to confirm my signing of the OCA.) > > @mruzicka once Dalibor connects your OCA with your GitHub account, this > review will be able to proceed. Do you have a test program that shows the bug? Also, can you take a look at whether you could turn the test case into an automated test? PR: https://git.openjdk.java.net/jfx/pull/16