RFR: 8231372: Correctly terminate secondary event loop in JFXPanel.setScene()

[Kevin Rushforth]

On Mon, 21 Oct 2019 22:52:47 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

> On Thu, 17 Oct 2019 15:28:31 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
> 
>> On Thu, 17 Oct 2019 15:28:30 GMT, mruzicka <github.com+562920+mruzicka at openjdk.org> wrote:
>> 
>>> On Thu, 17 Oct 2019 15:28:28 GMT, mruzicka <github.com+562920+mruzicka at 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?

@mruzicka - are you interested in moving this forward? If so, there is a pending request for a test case that needs to be answered before the review can proceed.

PR: https://git.openjdk.java.net/jfx/pull/16