Request for review: 7154048 [macosx] At least drag twice, the toolbar can be dragged to the left side. (original) (raw)

Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Wed Apr 18 08:40:54 PDT 2012


Please review a fix for CR 7154048. http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7154048

webrev: http://cr.openjdk.java.net/~alexsch/7154048/webrev.00/

Let's see the following test case:

The current JDK 8 implementation shows the following events on Mac OS X:

mouse pressed: javax.swing.JLabel mouse exited: javax.swing.JLabel mouse entered: javax.swing.JLabel mouse dragged: javax.swing.JLabel mouse exited: javax.swing.JLabel mouse entered: javax.swing.JButton mouse dragged: javax.swing.JLabel mouse exited: javax.swing.JButton mouse entered: Drag Window mouse exited: Drag Window mouse entered: javax.swing.JButton mouse released: javax.swing.JButton

There are several issues:

  1. The window does not receive the mouse entered event when it is created under the mouse
  2. There are JLabel exited/JButton entered events during the window dragging
  3. JLabel does not receive the mouse released event

The fix synthesizes the mouse entered/exited events manually if they are not received.

The entered/exited events synthesizing is added to setFrame, toFront, toBack, and zoom methods of the AWTWindow and CWrapper classes.

There is an option to add the events synthesizing to the windowDidResize notification. However this notification is sent when a window size is changed in both cases, programmatically and when user is resized the window. So in a lot of case there is no need for the our use case events generation.

The LWWindowPeer class is updated to not generate extra mouse enter/exit events during the mouse dragging.

Tho automated tests are added.

Thanks, Alexandr.



More information about the macosx-port-dev mailing list