All Popup overloads call dismiss on Esc key by default (#712)

* Expose root modifier instead of key events

* Use only one element for key input

* Handle Esc by default in existing Popup

* Create onKeyEvent modifier only if needed

* Revert EmptySemanticsElement

* Remove handling Esc key explicitly

* Add dismiss Popup by Esc tests

* Move dismissOnEsc to separate modifier

compose/material3/material3/src/skikoMain/kotlin/androidx/compose/material3/SkikoMenu.skiko.kt

@@ -124,7 +124,7 @@ fun DropdownMenu(
             onDismissRequest = onDismissRequest,
             popupPositionProvider = popupPositionProvider,
             onKeyEvent = {
-                handlePopupOnKeyEvent(it, onDismissRequest, focusManager!!, inputModeManager!!)
+                handlePopupOnKeyEvent(it, focusManager!!, inputModeManager!!)
             },
         ) {
             focusManager = LocalFocusManager.current
@@ -195,21 +195,17 @@ fun DropdownMenuItem(
 @ExperimentalComposeUiApi
 private fun handlePopupOnKeyEvent(
     keyEvent: KeyEvent,
-    onDismissRequest: () -> Unit,
     focusManager: FocusManager,
     inputModeManager: InputModeManager
 ): Boolean {
-    return if (keyEvent.type == KeyEventType.KeyDown && keyEvent.key == Key.Escape) {
-        onDismissRequest()
-        true
-    } else if (keyEvent.type == KeyEventType.KeyDown) {
-        when {
-            keyEvent.key == Key.DirectionDown -> {
+    return if (keyEvent.type == KeyEventType.KeyDown) {
+        when (keyEvent.key) {
+            Key.DirectionDown -> {
                 inputModeManager.requestInputMode(InputMode.Keyboard)
                 focusManager.moveFocus(FocusDirection.Next)
                 true
             }
-            keyEvent.key == Key.DirectionUp -> {
+            Key.DirectionUp -> {
                 inputModeManager.requestInputMode(InputMode.Keyboard)
                 focusManager.moveFocus(FocusDirection.Previous)
                 true