132974 2014-05-15 16:12:55 -0700 [iOS] www.wikipedia.org is loading much slower due to FormClient::willBeginInputSession 2015-03-16 17:23:35 -0700 1 1 1 Unclassified WebKit WebKit2 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 mitz mitz enrica jberkman oldest_to_newest 1009651 0 mitz 2014-05-15 16:12:55 -0700 <rdar://problem/16844952> The Wikipedia homepage contains an auto-focusing input element. When the element is focused, we tell the form bundle client that we’re starting a form input session, even though we don’t need to, because non-user-initiated focusing doesn’t start an input session on the UI process side. 1009657 1 231541 mitz 2014-05-15 16:25:42 -0700 Created attachment 231541 Only call the FromClient function if the UI process will start an input session 1009660 2 231541 enrica 2014-05-15 16:44:32 -0700 Comment on attachment 231541 Only call the FromClient function if the UI process will start an input session Looks good to me. 1009661 3 mitz 2014-05-15 16:47:07 -0700 Fixed in <http://trac.webkti.org/r168916>. 1077655 4 jberkman 2015-03-16 17:14:40 -0700 It looks like this prevents any scripts from being able to show the keyboard. Our app embeds an html page, and has an interface similar to Keynote, in that tapping on an element presents a wireframe, and double-tapping begins editing. We handle the double-tap gesture my making a call to -[WKWebView evaluateJavaScript:completionHandler:], which ends up calling element.focus() on the appropriate node. I suspect that setting m_userIsInteracting = true in WebPage::runJavaScriptInMainFrame() would work around this in some cases, but I'm guessing that wouldn't work if we ended up calling .focus() in a timeout handler. Are there any other mechanisms that could be done to allow element focusing from JS? Maybe require interaction only while the page is loading or something? We've been looking forward to moving to WKWebView since last June but this behaviour has prevented us from making the switch. 1077657 5 mitz 2015-03-16 17:18:02 -0700 (In reply to comment #4) > It looks like this prevents any scripts from being able to show the keyboard. > > Our app embeds an html page, and has an interface similar to Keynote, in > that tapping on an element presents a wireframe, and double-tapping begins > editing. We handle the double-tap gesture my making a call to -[WKWebView > evaluateJavaScript:completionHandler:], which ends up calling > element.focus() on the appropriate node. > > I suspect that setting m_userIsInteracting = true in > WebPage::runJavaScriptInMainFrame() would work around this in some cases, > but I'm guessing that wouldn't work if we ended up calling .focus() in a > timeout handler. > > Are there any other mechanisms that could be done to allow element focusing > from JS? Maybe require interaction only while the page is loading or > something? We've been looking forward to moving to WKWebView since last June > but this behaviour has prevented us from making the switch. Commenting on this resolved bug is not a good way to raise issues with the current behavior. If there are such issues, please report a new bug. Thanks! 1077661 6 jberkman 2015-03-16 17:23:35 -0700 n/p, filed bug 142757. Thanks! 231541 2014-05-15 16:25:42 -0700 2014-05-15 16:44:32 -0700 Only call the FromClient function if the UI process will start an input session bug-132974-20140515162518.patch text/plain 2300 mitz SW5kZXg6IFNvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi S2l0Mi9DaGFuZ2VMb2cJKHJldmlzaW9uIDE2ODkxNCkKKysrIFNvdXJjZS9XZWJLaXQyL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE2IEBACisyMDE0LTA1LTE1ICBEYW4gQmVy bnN0ZWluICA8bWl0ekBhcHBsZS5jb20+CisKKyAgICAgICAgW2lPU10gd3d3Lndpa2lwZWRpYS5v cmcgaXMgbG9hZGluZyBtdWNoIHNsb3dlciBkdWUgdG8gRm9ybUNsaWVudDo6d2lsbEJlZ2luSW5w dXRTZXNzaW9uCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p ZD0xMzI5NzQKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAg ICAqIFdlYlByb2Nlc3MvV2ViUGFnZS9pb3MvV2ViUGFnZUlPUy5tbToKKyAgICAgICAgKFdlYktp dDo6V2ViUGFnZTo6ZWxlbWVudERpZEZvY3VzKTogRG9u4oCZdCBjYWxsIEZvcm1DbGllbnQ6Ondp bGxCZWdpbklucHV0U2Vzc2lvbiBpZiB0aGUKKyAgICAgICAgZm9jdXMgaXNu4oCZdCB1c2VyLWlu aXRpYXRlZCwgYmVjYXVzZSBpbiB0aGF0IGNhc2UgdGhlIFVJIHByb2Nlc3Mgd2lsbCBiYWlsIG91 dCBvZgorICAgICAgICAtX3N0YXJ0QXNzaXN0aW5nTm9kZTp1c2VySXNJbnRlcmFjdGluZzp1c2Vy T2JqZWN0OiBiZWZvcmUgY3JlYXRpbmcgYW4gaW5wdXQgc2Vzc2lvbiBhbmQKKyAgICAgICAgY2Fs bGluZyB0aGUgLV93ZWJWaWV3OmRpZFN0YXJ0SW5wdXRTZXNzaW9uOi4KKwogMjAxNC0wNS0xNSAg QmVuamFtaW4gUG91bGFpbiAgPGJwb3VsYWluQGFwcGxlLmNvbT4KIAogICAgICAgICBbaU9TXVtX SzJdIFdLV2ViVmlldyBzZW5kcyBpbnZhbGlkIGNvbnRlbnQgb2Zmc2V0cyBhdCB0aGUgZW5kIG9m IGFuaW1hdGVkIHJlc2l6ZQpJbmRleDogU291cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9XZWJQYWdl L2lvcy9XZWJQYWdlSU9TLm1tCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJLaXQyL1dlYlByb2Nl c3MvV2ViUGFnZS9pb3MvV2ViUGFnZUlPUy5tbQkocmV2aXNpb24gMTY4ODk3KQorKysgU291cmNl L1dlYktpdDIvV2ViUHJvY2Vzcy9XZWJQYWdlL2lvcy9XZWJQYWdlSU9TLm1tCSh3b3JraW5nIGNv cHkpCkBAIC0xODc2LDcgKzE4NzYsMTQgQEAgdm9pZCBXZWJQYWdlOjplbGVtZW50RGlkRm9jdXMo V2ViQ29yZTo6TgogICAgICAgICBBc3Npc3RlZE5vZGVJbmZvcm1hdGlvbiBpbmZvcm1hdGlvbjsK ICAgICAgICAgZ2V0QXNzaXN0ZWROb2RlSW5mb3JtYXRpb24oaW5mb3JtYXRpb24pOwogICAgICAg ICBSZWZQdHI8QVBJOjpPYmplY3Q+IHVzZXJEYXRhOwotICAgICAgICBtX2Zvcm1DbGllbnQtPndp bGxCZWdpbklucHV0U2Vzc2lvbih0aGlzLCB0b0VsZW1lbnQobm9kZSksIFdlYkZyYW1lOjpmcm9t Q29yZUZyYW1lKCpub2RlLT5kb2N1bWVudCgpLmZyYW1lKCkpLCB1c2VyRGF0YSk7CisKKyAgICAg ICAgLy8gRklYTUU6IFdlIGNoZWNrIG1fdXNlcklzSW50ZXJhY3Rpbmcgc28gdGhhdCB3ZSBkb24n dCBiZWdpbiBhbiBpbnB1dCBzZXNzaW9uIGZvciBhCisgICAgICAgIC8vIHByb2dyYW1tYXRpYyBm b2N1cyB0aGF0IGRvZXNuJ3QgY2F1c2UgdGhlIGtleWJvYXJkIHRvIGFwcGVhci4gQnV0IHRoaXMg bWlzc2VzIHRoZSBjYXNlIG9mCisgICAgICAgIC8vIGEgcHJvZ3JhbW1hdGljIGZvY3VzIGhhcHBl bmluZyB3aGlsZSB0aGUga2V5Ym9hcmQgaXMgYWxyZWFkeSBzaG93bi4gT25jZSB3ZSBoYXZlIGEg d2F5IHRvCisgICAgICAgIC8vIGtub3cgdGhlIGtleWJvYXJkIHN0YXRlIGluIHRoZSBXZWIgUG9y Y2Vzcywgd2Ugc2hvdWxkIHJlZmluZSB0aGUgY29uZGl0aW9uLgorICAgICAgICBpZiAobV91c2Vy SXNJbnRlcmFjdGluZykKKyAgICAgICAgICAgIG1fZm9ybUNsaWVudC0+d2lsbEJlZ2luSW5wdXRT ZXNzaW9uKHRoaXMsIHRvRWxlbWVudChub2RlKSwgV2ViRnJhbWU6OmZyb21Db3JlRnJhbWUoKm5v ZGUtPmRvY3VtZW50KCkuZnJhbWUoKSksIHVzZXJEYXRhKTsKKwogICAgICAgICBzZW5kKE1lc3Nh Z2VzOjpXZWJQYWdlUHJveHk6OlN0YXJ0QXNzaXN0aW5nTm9kZShpbmZvcm1hdGlvbiwgbV91c2Vy SXNJbnRlcmFjdGluZywgSW5qZWN0ZWRCdW5kbGVVc2VyTWVzc2FnZUVuY29kZXIodXNlckRhdGEu Z2V0KCkpKSk7CiAgICAgfQogfQo=