GTK added a "KEY_" prefix to every key names. The support for GTK3 has been added with this commit https://gitlab.gnome.org/GNOME/gtk/commit/913cdf3be750a1e74c09b20edf55a57f9a919fcc and for GTK2 (from 2.21.8) with this commit https://gitlab.gnome.org/GNOME/gtk/commit/750c81f43dda6c783372b983e630ecd30b776d7e While on GTK2 and GTK3 it is still possible to use the old name scheme, on GTK4 only the new one is available. So it should be better to stick with the new name convention to ease a future GTK4 porting. I'm going to attach a patch to this issue.
Created attachment 370836 [details] Patch
Attachment 370836 [details] did not pass style-queue: ERROR: Source/WebCore/platform/gtk/PlatformKeyboardEventGtk.cpp:779: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] ERROR: Source/WebCore/platform/gtk/PlatformKeyboardEventGtk.cpp:1300: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] ERROR: Source/WebCore/ChangeLog:8: You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible. [changelog/nonewtests] [5] Total errors found: 3 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 370837 [details] Patch
Comment on attachment 370837 [details] Patch Thanks!
Comment on attachment 370837 [details] Patch Clearing flags on attachment: 370837 Committed r245849: <https://trac.webkit.org/changeset/245849>
All reviewed patches have been landed. Closing bug.