Bug 193876 – [iOS] Right command key has wrong value for property code

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit Misc.

[Daniel Bates]

Steps to reproduce

1. Visit <http://w3c.github.io/uievents/tools/key-event-viewer.html>
2. Focus the text field
3. Press the Command key on the right side of the keyboard.

The value of the code property is Unidentified. But it should be MetaRight. Compare to Mac.

[Radar WebKit Bug Importer]

<rdar://problem/47577308>

[Daniel Bates]

Created attachment 369523 [details]
Patch

[Daniel Bates]

Comment on attachment 369523 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=369523&action=review

> Source/WebCore/ChangeLog:9
> +        We're are looking for the wrong Windows virtual key code for the right command key.

We're are => We're

[Brent Fulgham]

Comment on attachment 369523 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=369523&action=review

r=me

> Source/WebCore/ChangeLog:11
> +        the correct value for the code property of the DOM key event.

Do we still need RWIN if the user is running a Microsoft keyboard?

> Source/WebCore/platform/ios/PlatformEventFactoryIOS.mm:278
> +    case VK_APPS: return "MetaRight"_s;

I wonder if we should just keep both of them?

[Daniel Bates]

(In reply to Brent Fulgham from comment #4)
> Comment on attachment 369523 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=369523&action=review
> 
> r=me
> 
> > Source/WebCore/ChangeLog:11
> > +        the correct value for the code property of the DOM key event.
> 
> Do we still need RWIN if the user is running a Microsoft keyboard?
> 

I don't know. This is iOS code. 😀

> > Source/WebCore/platform/ios/PlatformEventFactoryIOS.mm:278
> > +    case VK_APPS: return "MetaRight"_s;
> 
> I wonder if we should just keep both of them?

This is iOS code. 😀

[Daniel Bates]

Created attachment 369527 [details]
To land

[Daniel Bates]

Comment on attachment 369527 [details]
To land

Clearing flags on attachment: 369527

Committed r245162: <https://trac.webkit.org/changeset/245162>

[Daniel Bates]

All reviewed patches have been landed.  Closing bug.

[Daniel Bates]

(In reply to Brent Fulgham from comment #4)
> Comment on attachment 369523 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=369523&action=review
> 
> r=me
> 
> > Source/WebCore/ChangeLog:11
> > +        the correct value for the code property of the DOM key event.
> 
> Do we still need RWIN if the user is running a Microsoft keyboard?
> 

Oh I misread your comment. Anyway, the answer is No. These are Windows virtual key codes. How they map is entirely up to us (WebKit). iOS does not emit them. So, I chose to match Mac which maps the key code for the right command to VK_APPS.

> > Source/WebCore/platform/ios/PlatformEventFactoryIOS.mm:278
> > +    case VK_APPS: return "MetaRight"_s;
> 
> I wonder if we should just keep both of them?

No need to keep both. I defined the mapping!