Bug 197746 – [iOS] Lazily request keyboard on first hardware keydown when a non-editable element is focused

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

[Daniel Bates]

In bug #197745 we made keyboard instantiation lazy: we always create one if a hardware keyboard is attached OR a content editable element is focused. We can be even lazier! Specifically the first disjunct can be even lazier. So, I think we only need to create a keyboard if any of the following are satisfied:

a. a non-editable element is focused AND we receive *one* hardware keydown event.
   ii. Take care to reset this on hardware keyboard availability changes. Side note, we could even reduce memory footprint here by tearing down the keyboard here when switching from attached to detached ONLY in the non-editable element case.
b. an editable element is focused (obvious, right?)

[Radar WebKit Bug Importer]

<rdar://problem/50630406>

[Daniel Bates]

Created attachment 369511 [details]
Patch

[Daniel Bates]

Comment on attachment 369511 [details]
Patch

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

> Source/WebKit/ChangeLog:10
> +        Be even lazier. We only need to create a keyboard if an editable element is focused or on
> +        the first hardware keydown when a non-editable element is focused.

Note that we already do the former. This patch is about the latter.

[Daniel Bates]

Created attachment 369512 [details]
Patch

[EWS Watchlist]

Attachment 369512 [details] did not pass style-queue:


ERROR: Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:1673:  Line contains only semicolon. If this should be an empty statement, use { } instead.  [whitespace/semicolon] [5]
Total errors found: 1 in 3 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Brent Fulgham]

Comment on attachment 369512 [details]
Patch

r=me

[Daniel Bates]

Created attachment 369519 [details]
To Land

[Daniel Bates]

Comment on attachment 369519 [details]
To Land

Clearing flags on attachment: 369519

Committed r245154: <https://trac.webkit.org/changeset/245154>

[Daniel Bates]

All reviewed patches have been landed.  Closing bug.