# HG changeset patch # User serb # Date 1488461093 -10800 # Node ID 35def8dd7581e5eaa81ae09675c7719b1af73dc1 # Parent 443a141b81c6f3e719202f7a3f922ab0e2d68cb5 8176009: The awt robot use incorrect location in a multi-screen environment Reviewed-by: azvegint, alexsch diff -r 443a141b81c6 -r 35def8dd7581 jdk/src/java.desktop/unix/classes/sun/awt/X11/XRobotPeer.java --- a/jdk/src/java.desktop/unix/classes/sun/awt/X11/XRobotPeer.java Thu Mar 02 13:32:30 2017 +0300 +++ b/jdk/src/java.desktop/unix/classes/sun/awt/X11/XRobotPeer.java Thu Mar 02 16:24:53 2017 +0300 @@ -33,7 +33,6 @@ import sun.awt.SunToolkit; import sun.awt.UNIXToolkit; import sun.awt.X11GraphicsConfig; -import sun.awt.X11GraphicsEnvironment; class XRobotPeer implements RobotPeer { @@ -77,14 +76,7 @@ @Override public void mouseMove(int x, int y) { - X11GraphicsEnvironment x11ge = (X11GraphicsEnvironment) - GraphicsEnvironment.getLocalGraphicsEnvironment(); - if(x11ge.runningXinerama()) { - Rectangle sb = xgc.getBounds(); - mouseMoveImpl(xgc, xgc.scaleUp(x + sb.x), xgc.scaleUp(y + sb.y)); - } else { - mouseMoveImpl(xgc, xgc.scaleUp(x), xgc.scaleUp(y)); - } + mouseMoveImpl(xgc, xgc.scaleUp(x), xgc.scaleUp(y)); } @Override diff -r 443a141b81c6 -r 35def8dd7581 jdk/test/java/awt/Robot/MultiScreenRobotPosition/MultiScreenRobotPosition.java --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/jdk/test/java/awt/Robot/MultiScreenRobotPosition/MultiScreenRobotPosition.java Thu Mar 02 16:24:53 2017 +0300 @@ -0,0 +1,75 @@ +/* + * Copyright (c) 2017, Oracle and/or its affiliates. All rights reserved. + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * + * This code is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License version 2 only, as + * published by the Free Software Foundation. + * + * This code is distributed in the hope that it will be useful, but WITHOUT + * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or + * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License + * version 2 for more details (a copy is included in the LICENSE file that + * accompanied this code). + * + * You should have received a copy of the GNU General Public License version + * 2 along with this work; if not, write to the Free Software Foundation, + * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. + * + * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA + * or visit www.oracle.com if you need additional information or have any + * questions. + */ + +import java.awt.Frame; +import java.awt.GraphicsDevice; +import java.awt.GraphicsEnvironment; +import java.awt.Rectangle; +import java.awt.Robot; +import java.awt.event.MouseAdapter; +import java.awt.event.MouseEvent; + +/** + * @test + * @bug 8176009 + */ +public class MultiScreenRobotPosition { + + private static volatile boolean fail = true; + + public static void main(String[] args) throws Exception { + GraphicsDevice[] sds = GraphicsEnvironment.getLocalGraphicsEnvironment() + .getScreenDevices(); + for (final GraphicsDevice gd : sds) { + fail = true; + Robot robot = new Robot(gd); + robot.setAutoDelay(100); + robot.setAutoWaitForIdle(true); + + Frame frame = new Frame(gd.getDefaultConfiguration()); + frame.setUndecorated(true); + frame.setSize(400, 400); + frame.setVisible(true); + robot.waitForIdle(); + + frame.addMouseListener(new MouseAdapter() { + @Override + public void mouseClicked(MouseEvent e) { + System.out.println("e = " + e); + fail = false; + } + }); + + Rectangle bounds = frame.getBounds(); + robot.mouseMove(bounds.x + bounds.width / 2, + bounds.y + bounds.height / 2); + robot.mousePress(MouseEvent.BUTTON1_DOWN_MASK); + robot.mouseRelease(MouseEvent.BUTTON1_DOWN_MASK); + frame.dispose(); + if (fail) { + System.err.println("Frame bounds = " + bounds); + throw new RuntimeException("Click in the wrong location"); + } + } + } +}