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
