I'll defer to Phil on this review.I have no concerns over the fix, and can confirm that the failing tests now pass on my system.
-- Kevin Laurent Bourgès wrote:
Kevin, Please review this webrev that fixes the NPE in getMediaName(): JBS: https://bugs.openjdk.java.net/browse/JDK-8196617webrev: http://cr.openjdk.java.net/~lbourges/fx/fx-8196617.0/ <http://cr.openjdk.java.net/%7Elbourges/fx/fx-8196617.0/>It was tested OK on the appveyor CI that failed before the patch. Thanks, Laurent