RFR: 8307934 JRobot.moveMouseTo must access component on EDT

Renjith Kannath Pariyangad duke at openjdk.org
Tue Jun 20 06:32:05 UTC 2023


On Thu, 15 Jun 2023 22:53:56 GMT, Phil Race <prr at openjdk.org> wrote:

>> Thanks @prrace for the reviews, 
>> I have added EDT support for 'getCenterPoint' and 'convertRectToScreen' functions, executed all the tests which are using this function and confirmed its working.
>
> But you seem to have ignored my comment about documenting it on 'all entry points' meaning EVERY exported API from this class which can end up here should say something like
> "This method will invoke code on the Event Dispatch Thread".

Sorry I didn't ignored your valuable comment regarding documentation.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/14354#discussion_r1234794697



More information about the client-libs-dev mailing list