On Wed, 4 Dec 2019 14:50:19 GMT, Robert Lichtenberger <rlich...@openjdk.org> 
wrote:

> The pull request has been updated with additional changes.
> 
> ----------------
> 
> Added commits:
>  - 7c5cf198: 8232524: Test cleanup: terminate background thread upon failure.
> 
> Changes:
>   - all: https://git.openjdk.java.net/jfx/pull/17/files
>   - new: https://git.openjdk.java.net/jfx/pull/17/files/7e80839f..7c5cf198
> 
> Webrevs:
>  - full: https://webrevs.openjdk.java.net/jfx/17/webrev.01
>  - incr: https://webrevs.openjdk.java.net/jfx/17/webrev.00-01
> 
>   Issue: https://bugs.openjdk.java.net/browse/JDK-8232524
>   Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod
>   Patch: https://git.openjdk.java.net/jfx/pull/17.diff
>   Fetch: git fetch https://git.openjdk.java.net/jfx pull/17/head:pull/17

Looks good. I can confirm that the new tests fail without the fix and pass with 
the fix.

----------------

Approved by kcr (Lead).

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

Reply via email to