Summary: | [iOS] No DOM keypress event emitted for cursor key commands when non-editable element is focused | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Daniel Bates <dbates> | ||||||
Component: | WebKit Misc. | Assignee: | Daniel Bates <dbates> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | megan_gardner, ryanhaddad, webkit-bug-importer, wenson_hsieh | ||||||
Priority: | P2 | Keywords: | InRadar, PlatformOnly | ||||||
Version: | WebKit Local Build | ||||||||
Hardware: | iPhone / iPad | ||||||||
OS: | iOS 12 | ||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=193634 | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 190571 | ||||||||
Attachments: |
|
Description
Daniel Bates
2019-03-12 09:42:22 PDT
Created attachment 364404 [details]
Patch and tests
Bots will likely be red. Will fix up patch based on the bots. Comment on attachment 364404 [details] Patch and tests View in context: https://bugs.webkit.org/attachment.cgi?id=364404&action=review > Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:2725 > + // UIKit IPI do not forward to the WebView. As we chatted about earlier, this could read a bit nicer (e.g. "These are UIKit IPI methods, so we don't forward them to the web view.") (In reply to Daniel Bates from comment #3) > Bots will likely be red. Will fix up patch based on the bots. Nope, they won't because we skip fast/events/ios on OpenSource per <https://trac.webkit.org/browser/trunk/LayoutTests/TestExpectations?rev=242724#L37> :D and no longer mark this directory as Pass in any iOS TestExpectation file o_O. That works out well for me and the tests in this patch since they depend on UIKit fixes. However, that appears to be a regressions from <https://trac.webkit.org/changeset/190368/webkit#file7>. (In reply to Daniel Bates from comment #5) > (In reply to Daniel Bates from comment #3) > > Bots will likely be red. Will fix up patch based on the bots. > > Nope, they won't because we skip fast/events/ios on OpenSource per > <https://trac.webkit.org/browser/trunk/LayoutTests/ > TestExpectations?rev=242724#L37> :D and no longer mark this directory as > Pass in any iOS TestExpectation file o_O. That works out well for me and the > tests in this patch since they depend on UIKit fixes. However, that appears > to be a regressions from > <https://trac.webkit.org/changeset/190368/webkit#file7>. The tests were timing out on so we removed the PASS entry for fast/events/ios from the iOS TestExpectation file in <https://trac.webkit.org/changeset/194059/webkit> (bug #152134). So, now we do not run these tests at all, at least on the OpenSource bots. Created attachment 365031 [details]
To land
Committed r243082: <https://trac.webkit.org/changeset/243082> |