Thank you for the review, Erik! I will update the fix as you suggested before push if you do not mind.
Thanks, Dmitry > On 25 Oct 2019, at 14:00, Erik Joelsson <erik.joels...@oracle.com> wrote: > > Looks good. I would put in a "the" before "operating system". > > /Erik > > On 2019-10-25 03:27, Dmitry Markov wrote: >> Hello, >> >> Could you review the fix for jdk14, please? >> >> bug: https://bugs.openjdk.java.net/browse/JDK-8232880 >> <https://bugs.openjdk.java.net/browse/JDK-8232880> >> webrev: http://cr.openjdk.java.net/~dmarkov/8232880/webrev.00/ >> <http://cr.openjdk.java.net/~dmarkov/8232880/webrev.00/> >> >> Some Client UI tests use the following key sequence to show/hide tooltip >> message: “CTRL” + “F1". However this key combination is reserved by >> operating system on OS X platform. As a results the tests fail. >> Test documentation should be updated with some notes regarding this. >> >> Thanks, >> Dmitry >>