Disable keyboard input (with exceptions) in full-screen mode.

• Component: New Bugs
• Status: RESOLVED FIXED

[Jer Noble]

Disable keyboard input (with exceptions) in full-screen mode.

[Jer Noble]

<rdar://problem/9450785>

[Jer Noble]

Created attachment 93740 [details]
Patch

[Darin Adler]

Comment on attachment 93740 [details]
Patch

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

> Source/WebCore/page/EventHandler.cpp:2442
> +    if ((keyCode >= VK_BACK && keyCode <= VK_CAPITAL)
> +        || (keyCode >= VK_SPACE && keyCode <= VK_DELETE)
> +        || (keyCode >= VK_OEM_1 && keyCode <= VK_OEM_PLUS)
> +        || (keyCode >= VK_MULTIPLY && keyCode <= VK_OEM_8))
> +        return true;
> +
> +    return false;

This seems quite unclear. Is this specific set of key code ranges straight out of some specification? If not, what is the rationale for what keys are listed here and what keys are not?

Also, if (x) return true; return false; is an anti-pattern. Just use a return.

[Jer Noble]

(In reply to comment #3)
> (From update of attachment 93740 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=93740&action=review
> 
> > Source/WebCore/page/EventHandler.cpp:2442
> > +    if ((keyCode >= VK_BACK && keyCode <= VK_CAPITAL)
> > +        || (keyCode >= VK_SPACE && keyCode <= VK_DELETE)
> > +        || (keyCode >= VK_OEM_1 && keyCode <= VK_OEM_PLUS)
> > +        || (keyCode >= VK_MULTIPLY && keyCode <= VK_OEM_8))
> > +        return true;
> > +
> > +    return false;
> 
> This seems quite unclear. Is this specific set of key code ranges straight out of some specification? If not, what is the rationale for what keys are listed here and what keys are not?

Yes, it's specifically from the proposed Full Screen API:

"Toplevel browsing contexts can be in a "keys disabled" state. In this state, the user agent must suppress all keyup, keydown and keypress events whose keyCode is not in one of the following ranges:

DOM_VK_CANCEL to DOM_VK_CAPS_LOCK, inclusive
DOM_VK_SPACE to DOM_VK_DELETE, inclusive
DOM_VK_SEMICOLON to DOM_VK_EQUALS, inclusive
DOM_VK_MULTIPLY to DOM_VK_META, inclusive"

The key codes listed in the patch are the equivalents to the Mozilla-specific enums in the spec.

> Also, if (x) return true; return false; is an anti-pattern. Just use a return.

Sure thing.  Thanks!

[Jer Noble]

Committed r86741: <http://trac.webkit.org/changeset/86741>