[Webkit-unassigned] [Bug 228802] REGRESSION (r280492): Clicking backspace on sign in fields results in deleting two characters instead of one

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Aug 4 15:57:22 PDT 2021


--- Comment #4 from Wenson Hsieh <wenson_hsieh at apple.com> ---
(In reply to Wenson Hsieh from comment #3)
> (In reply to Tim Horton from comment #2)
> > Comment on attachment 434942 [details]
> > Patch
> > 
> > View in context:
> > https://bugs.webkit.org/attachment.cgi?id=434942&action=review
> > 
> > > Source/WebCore/ChangeLog:8
> > > +        No tests yet.
> > 
> > The fact that this didn't break any tests is astounding; you should see if
> > Wenson has any good ideas about how to test the software keyboard codepath
> > (we may not have a good way, and no need for making one to block the
> > regression fix).
> The last time I looked into adding support for this, I did not find any
> straightforward way to achieve this (using API, SPI, or even IPI). That
> said, it has been several years since this investigation, so we should take
> another look at this!
> (In the shorter term, perhaps it's also possible to write an API test that
> swizzles/mocks/calls directly into `WKContentView` to simulate pressing the
> delete key on the software keyboard on iOS).

>From a quick debugging session on my iPhone, it looks like we might be able to grab the TIKeyboardLayout from the active UIKeyboardImpl, look up the frame (CGRect) of the corresponding key string, map that from input view coordinates into screen coordinates, and then synthesize HID events to tap the center of that rect.

I suppose any test that uses this mechanism would need to listen for DOM events (keyup) to know that the key press is finished.

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20210804/944d75c9/attachment-0001.htm>

More information about the webkit-unassigned mailing list