RESOLVED FIXED 212848
[WebDriver] Allow platform-dependent modifier keys conversion to WebEvent::Modifier 
https://bugs.webkit.org/show_bug.cgi?id=212848
Summary [WebDriver] Allow platform-dependent modifier keys conversion to WebEvent::Mo...
Lauro Moura
Reported 2020-06-05 14:41:15 PDT
Currently, WebAutomationSession::simulateMouseInteraction converts m_currentModifiers (which stores the current native platform modifiers) directly to OptionSet<WebEvent::Modifier> using OptionSet::fromRaw(m_currentModifiers). This causes problems as the fields in the native mappings is completely different from the WebEvent ones: |Key | WebEvent | WPE | GTK | Mac | |--------|----------|-----|-----|-----| |Shift | 0 | 1 | 0 | 17 | |Control | 1 | 0 | 2 | 18 | |Alt | 2 | 2 | 28 | 19 | |Meta | 3 | 3 | X | 20 | |Caps | 4 | X | 1 | 16 | |--------|----------|-----|-----|-----| For example, the current GTK context menu tests tests failing with the ctrl key pressed.
Attachments
Patch (7.36 KB, patch)
2020-06-05 14:48 PDT, Lauro Moura 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Lauro Moura
Comment 1 2020-06-05 14:48:51 PDT
Created attachment 401202 [details] Patch
Blaze Burg
Comment 2 2020-06-05 16:08:17 PDT
Comment on attachment 401202 [details] Patch r=me, good catch.
EWS
Comment 3 2020-06-07 17:34:38 PDT
Committed r262703: <https://trac.webkit.org/changeset/262703> All reviewed patches have been landed. Closing bug and clearing flags on attachment 401202 [details].
Radar WebKit Bug Importer
Comment 4 2020-06-07 17:35:18 PDT
<rdar://problem/64098495>
Note You need to log in before you can comment on or make changes to this bug.