Fix key event used for close requests

[domenic]

Esc keydown has always been what triggered close request behavior. This was just a mistake in writing the spec. You can verify interactively using https://boom-bath.glitch.me/dialog-events.html

/cc @lukewarlow @josepharhar

• At least two implementers are interested (and none opposed):
  • This is already what everyone implements for dialog and fullscreen, and what Chromium implements for CloseWatcher
• Tests are written and can be reviewed and commented upon at:
  • CloseWatcher: fix Esc key tests web-platform-tests/wpt#44927
• Implementation bugs are filed:
  • Chromium: https://bugs.chromium.org/p/chromium/issues/detail?id=1512914
  • Gecko: N/A, covered by general close requests bug
  • WebKit: N/A, covered by general close requests bug
  • Deno (only for timers, structured clone, base64 utils, channel messaging, module resolution, web workers, and web storage): N/A
  • Node.js (only for timers, structured clone, base64 utils, channel messaging, and module resolution): N/A
• MDN issue is filed: Not necessary
• The top of this comment includes a clear commit message to use.

(See WHATWG Working Mode: Changes for more details.)

---

/interaction.html ( diff )

[keithamus]

LGTM 👍

[domenic]

Finally was able to dig into the tests issue here, and I found the problem. Tests will be exported from https://chromium-review.googlesource.com/c/chromium/src/+/5339289 soon.