[7u40] Review request for 8019628: [macosx] closed/java/awt/Modal/BlockedMouseInputTest/BlockedMouseInputTest.html failed since 7u40b30 on MacOS (original) (raw)
Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Fri Jul 5 06:47:51 PDT 2013
- Previous message: Java 1.6.0_51 regressions, somewhere we can download an alternative ?
- Next message: [7u40] Review request for 8019628: [macosx] closed/java/awt/Modal/BlockedMouseInputTest/BlockedMouseInputTest.html failed since 7u40b30 on MacOS
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Hello,
Could you review the fix for the regression in JDK 7u40: bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8019628 webrev: http://cr.openjdk.java.net/~alexsch/8019628/webrev.00
Generated enter/exit mouse events have been changed to use the current mouse event id during the last merge in the JDK 7u40: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/diff/9062acbc098c/src/macosx/classes/sun/lwawt/LWWindowPeer.java For example
Point oldp = lastMouseEventPeer.windowToLocal(x, y, this);
postEvent(new MouseEvent(lastMouseEventPeer.getTarget(),
MouseEvent.MOUSE_EXITED,
when, modifiers,
oldp.x, oldp.y, screenX, screenY,
clickCount, popupTrigger, button));
Component target = lastMouseEventPeer.getTarget();
postMouseEnteredExitedEvent(target, id, when, modifiers, oldp,
screenX, screenY, clickCount, popupTrigger, button);
So the mouse enter/exit events are now can be lost and wrong events can be sent for the lastMouseEventPeer. The fix adds the explicit mouse enter/exit events declaration back as it was before the merge.
There are 2 failed tests in the JDK 7u repository because of the regression so it is not necessary to write new one.
Thanks, Alexandr.
- Previous message: Java 1.6.0_51 regressions, somewhere we can download an alternative ?
- Next message: [7u40] Review request for 8019628: [macosx] closed/java/awt/Modal/BlockedMouseInputTest/BlockedMouseInputTest.html failed since 7u40b30 on MacOS
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]