On Tue, 29 Jul 2025 21:40:46 GMT, Alisen Chung <[email protected]> wrote:
>> Some useful methods in ExtendedRobot should be migrated into Robot itself so >> that ExtendedRobot can be removed in the future. The tests using >> ExtendedRobot for these migrated methods are changed to use only Robot >> (removing unnecessary building of ExtendedRobot). > > Alisen Chung has updated the pull request incrementally with one additional > commit since the last revision: > > revert robot set/get for step delay/length and readd > glide(int,int,int,int,int,int), remove restriction on dist < stepLength and > corresponding IAE from spec src/java.desktop/share/classes/java/awt/Robot.java line 846: > 844: /** > 845: * A convenience method that moves the mouse in multiple > 846: * steps from its current location to the destination coordinates. > Invokes I think this is the text that caused Joe to write "From reading the text, it should like the method would benefit from an @implSpec tag describing the behavior." You can read about implSpec here https://bugs.openjdk.org/browse/JDK-8068562 ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22044#discussion_r2248945478
