Edge browser sends KeyEvent asynchronously

Edge:handleAcceleratorKeyPressed must handle Browser:sendKeyEvent
asynchronously as KeyEvent's Listeners can cause deadlock. Hence, the
Handled of this event is set if there exists a listener for the event or
a Display filter.

Author: amartya4256

bundles/org.eclipse.swt/Eclipse SWT Browser/win32/org/eclipse/swt/browser/Edge.java

@@ -1573,9 +1573,12 @@ int handleAcceleratorKeyPressed(long pView, long pArgs) {
 			return COM.S_OK;
 		}
 
-		if (!sendKeyEvent(keyEvent)) {
-			args.put_Handled(true);
-		}
+		asyncExec(() -> {
+			sendKeyEvent(keyEvent);
+		});
+
+		boolean hasListener = browser.getListeners(keyEvent.type).length != 0 || browser.filters(keyEvent.type);
+		args.put_Handled(hasListener);
 	} else {
 		keyEvent.type = SWT.KeyUp;
 		browser.notifyListeners (keyEvent.type, keyEvent);

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

@@ -870,6 +870,18 @@ NSRect expansionFrameWithFrame_inView(long id, long sel, NSRect cellRect, long v
 	return result;
 }
 
+/**
+ * Checks for if there exists a listener in the collection of listeners who will
+ * be notified when an event of the given type occurs anywhere in a widget.
+ *
+ * Needs to be public as the extended classes of widget should primarily use and
+ * these classes can be in different packages.
+ *
+ * @param eventType
+ * @return if there exists a listener which reacts to the eventType
+ *
+ * @since 3.133
+ */
 public boolean filters (int eventType) {
 	return display.filters (eventType);
 }

bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/Widget.java

@@ -1194,6 +1194,18 @@ long filterProc(long xEvent, long gdkEvent, long data2) {
 	return 0;
 }
 
+/**
+ * Checks for if there exists a listener in the collection of listeners who will
+ * be notified when an event of the given type occurs anywhere in a widget.
+ *
+ * Needs to be public as the extended classes of widget should primarily use and
+ * these classes can be in different packages.
+ *
+ * @param eventType
+ * @return if there exists a listener which reacts to the eventType
+ *
+ * @since 3.133
+ */
 public boolean filters (int eventType) {
 	return display.filters (eventType);
 }

bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/widgets/Widget.java

@@ -510,6 +510,18 @@ void error (int code) {
 	SWT.error(code);
 }
 
+/**
+ * Checks for if there exists a listener in the collection of listeners who will
+ * be notified when an event of the given type occurs anywhere in a widget.
+ *
+ * Needs to be public as the extended classes of widget should primarily use and
+ * these classes can be in different packages.
+ *
+ * @param eventType
+ * @return if there exists a listener which reacts to the eventType
+ *
+ * @since 3.133
+ */
 public boolean filters (int eventType) {
 	return display.filters (eventType);
 }