fix SWT table/tree control SelectionListener on Mac (#1079)

Fix SWT table/tree control SelectionListener on Mac

SelectionEvent.item in callback method widgetSelected now contains the
current selected item

Fixes: #1052

---------

Co-authored-by: Tobias Melcher <tobias.melcher@sap.com>

bundles/org.eclipse.swt/Eclipse SWT/cocoa/org/eclipse/swt/widgets/Table.java

@@ -3477,6 +3477,9 @@ void tableViewSelectionDidChange (long id, long sel, long aNotification) {
 
 @Override
 void tableViewSelectionIsChanging (long id, long sel, long aNotification) {
+	// tableViewSelectionIsChanging is called when pressing ARROW_DOWN, ARROW_UP key
+	// don't run sendSelection because it would then gather the "old" incorrect selected row
+	if (keyDown) return;
 	didSelect = true;
 	sendSelection();
 }

bundles/org.eclipse.swt/Eclipse SWT/cocoa/org/eclipse/swt/widgets/Tree.java

@@ -96,7 +96,7 @@ public class Tree extends Composite {
 
 	/* Used to control drop feedback when DND.FEEDBACK_EXPAND and DND.FEEDBACK_SCROLL is set/not set */
 	boolean shouldExpand = true, shouldScroll = true;
-
+	boolean keyDown;
 	final int nativeItemHeight;
 
 	static int NEXT_ID;
@@ -2108,7 +2108,9 @@ boolean isTrim (NSView view) {
 @Override
 void keyDown(long id, long sel, long theEvent) {
 	ignoreSelect = preventSelect = false;
+	keyDown = true;
 	super.keyDown(id, sel, theEvent);
+	keyDown = false;
 }
 
 @Override
@@ -2463,6 +2465,9 @@ void outlineViewSelectionDidChange (long id, long sel, long notification) {
 
 @Override
 void outlineViewSelectionIsChanging (long id, long sel, long notification) {
+	// outlineViewSelectionIsChanging is called when pressing ARROW_DOWN, ARROW_UP key
+	// don't run sendSelection because it would then gather the "old" incorrect selected row
+	if (keyDown) return;
 	didSelect = true;
 	sendSelection ();
 }