Hello Phil and Sergey,

The new version of the back out fix, which does not remove the regression test and adds "@ignore" jtreg tag, is following.

Webrev (the 2nd version): http://cr.openjdk.java.net/~alitvinov/8181192/jdk9/webrev.01

Thank you,
Anton

On 6/1/2017 10:09 PM, Sergey Bylokhov wrote:
The fix is fine but rather than deleting the test just mark it @ignore

.. it can be then updated
with a new version of the fix.
+1


-phil.

On 6/1/17, 11:19 AM, Anton Litvinov wrote:
Hello,

Could you please review the following fix for the bug.

Bug: https://bugs.openjdk.java.net/browse/JDK-8181192
Webrev:
http://cr.openjdk.java.net/~alitvinov/8181192/jdk9/webrev.00
The bug is a showstopper bug consisting in hanging during the call
to
"javafx.print.PrinterJob.showPrintDialog(Windows)" method in JavaFX
application. The bug is a regression of the fix for the bug
JDK-8167102.
The solution is a direct back out of the fix JDK-8167102, so it is a
reversed version of the patch for JDK-8167102.

Thank you,
Anton

Reply via email to