[8u] RFR: 8226806: [macOS 10.14] Methods of Java Robot should be called from appropriate thread

Andrew Hughes gnu.andrew at redhat.com
Wed Jun 9 17:18:24 UTC 2021


On 11:25 Wed 09 Jun     , Vladimir Kempik wrote:
> Hello
> Please review this backport of 8226806.
> It wasn’t applying cleanly due to context code changes, also smart part of newer testlibrary were needed.
> Since the support for xcode12 were approved ( see JDK-8267545 ), it becomes important to backport some of the bug fixes as well.
> This fix passed all the testing in zulu8 for past 3 releases ( since October 2020)
> 
> The webrev doesn’t reflect white space changes in CRobot.m, so the patch looks smaller than original.
> 
> The webrev: http://cr.openjdk.java.net/~vkempik/8226806/webrev.01/
> The bug: https://bugs.openjdk.java.net/browse/JDK-8226806
> The orignal patch: https://hg.openjdk.java.net/jdk/jdk/rev/23dc143cca50 <https://hg.openjdk.java.net/jdk/jdk/rev/23dc143cca50>
> 
> Thanks in advance, Vladimir.

Regarding bug fixes, would it be worth bringing in JDK-8155740 &
JDK-8156460? I believe these would remove the differences in the
keyEvent code.

The test library additions seem odd, given those methods are
already in jdk/test/lib/jdk/test/lib/process/OutputAnalyzer.java.
I don't know why we have two OutputAnalyzer.java files, but that
also seems to be true in 11u!

Thanks,
-- 
Andrew :)

Senior Free Java Software Engineer
OpenJDK Package Owner
Red Hat, Inc. (http://www.redhat.com)

PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222


More information about the jdk8u-dev mailing list