<AWT Dev> Review Request for 8039749: Migrate needed functionality from all subclasses of java.awt.Robot in jdk/test directory to the ancestor

Anthony Petrov anthony.petrov at oracle.com
Wed Aug 27 11:16:32 UTC 2014


Hi Dmitriy,

While I realize that all the new methods are useful when writing JDK 
regression tests, do you have any evidence that would suggest that these 
same methods could be useful to and/or have been requested by external 
developers? All of them look like convenient APIs and I'm not entirely 
convinced if we should ultimately add them all to our public Robot API. 
Personally, I don't see a problem with using the custom ExtendedRobot 
class specifically for tests. This also helps reduce the JDK and JRE 
static footprints, btw. But then again, I'm not an SQE engineer.

I don't have a strong opinion on this and I'm leaving the final decision 
to Sergey and other AWT team members, but I just thought I'd bring this 
up here.

As for the implementation, I see that you're adding realSync() calls in 
some places where they were not previously there. For example, calling 
Robot.waitForIdle() before the fix would not cause a realSync() to 
occur, while after your fix it does. This is a significant change from 
threading and native event queue synchronization perspectives, and I'm 
not sure if this should be done. Again, I do know that in tests it is 
useful to call realSync() here and there, but I'm not sure we should 
spread the calls all over the Robot implementation simply because of 
this reason. The calls may in fact produce some unwanted side-effects 
for existing applications employing the Robot (e.g. introducing unwanted 
delays, or performing excessive synchronization while the app didn't 
really need it, etc.) I consider this change very risky.

--
best regards,
Anthony

On 8/26/2014 5:40 PM, Dmitriy Ermashov wrote:
> Hi awt team!
>
> A few months ago we have consolidated methods required by functional and
> regression tests in ExtendedRobot class. After a period of extensive
> testing, it's time to migrate them to java.awt.Robot.
>
> Please review the changeset:
> http://cr.openjdk.java.net/~dermashov/8039749/webrev.00/
>
> Corresponding bug:
> https://bugs.openjdk.java.net/browse/JDK-8039749
>
> Note that this change is an important prerequisite to a massive
> regression and functional test suites change for Jigsaw.
> As one can see the webrev contains changes in class signature. The CCC
> request will take place after the code review.
>
> Thanks,
> Dima


More information about the awt-dev mailing list