| Summary: | [iOS] Spelling suggestions cannot be selected in focused form controls when zoomed in | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Wenson Hsieh <wenson_hsieh> | ||||||||||
| Component: | Forms | Assignee: | Wenson Hsieh <wenson_hsieh> | ||||||||||
| Status: | RESOLVED FIXED | ||||||||||||
| Severity: | Normal | CC: | bdakin, cdumez, commit-queue, dbates, megan_gardner, pxlcoder, thorton, webkit-bug-importer, wenson_hsieh | ||||||||||
| Priority: | P2 | Keywords: | InRadar | ||||||||||
| Version: | WebKit Nightly Build | ||||||||||||
| Hardware: | Unspecified | ||||||||||||
| OS: | Unspecified | ||||||||||||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=190562 | ||||||||||||
| Attachments: |
|
||||||||||||
|
Description
Wenson Hsieh
2018-07-27 12:12:37 PDT
Created attachment 345938 [details]
Patch
Comment on attachment 345938 [details] Patch Clearing flags on attachment: 345938 Committed r234436: <https://trac.webkit.org/changeset/234436> All reviewed patches have been landed. Closing bug. Comment on attachment 345938 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=345938&action=review > LayoutTests/fast/forms/ios/click-should-not-suppress-misspelling-expected.txt:3 > +TEST COMPLETE This output does not look correct. We’re missing the description and this line should be after all other lines. Comment on attachment 345938 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=345938&action=review >> LayoutTests/fast/forms/ios/click-should-not-suppress-misspelling-expected.txt:3 >> +TEST COMPLETE > > This output does not look correct. We’re missing the description and this line should be after all other lines. This was generated by running the layout test. I'll run it again to double check. Note that the description is only written when !window.testRunner as a courtesy to anyone running the test manually. Comment on attachment 345938 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=345938&action=review >>> LayoutTests/fast/forms/ios/click-should-not-suppress-misspelling-expected.txt:3 >>> +TEST COMPLETE >> >> This output does not look correct. We’re missing the description and this line should be after all other lines. > > This was generated by running the layout test. I'll run it again to double check. > > Note that the description is only written when !window.testRunner as a courtesy to anyone running the test manually. I just finished a clean build of trunk iOS simulator (using iOS 12), ran this test, and confirmed that it passes. (In reply to Wenson Hsieh from comment #7) > Comment on attachment 345938 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=345938&action=review > > >>> LayoutTests/fast/forms/ios/click-should-not-suppress-misspelling-expected.txt:3 > >>> +TEST COMPLETE > >> > >> This output does not look correct. We’re missing the description and this line should be after all other lines. > > > > This was generated by running the layout test. I'll run it again to double check. > > > > Note that the description is only written when !window.testRunner as a courtesy to anyone running the test manually. > > I just finished a clean build of trunk iOS simulator (using iOS 12), ran > this test, and confirmed that it passes. I didn’t mean to imply there is a correctness issue by my remark. I meant to say that the output of the test does not conform to the expected output of a test that uses js-test. In particular, a js-test should emit a description, plus a line that states that there will be a series of PASS messages, PASS/FAIL output, a successfully parsed line and finally a TEST COMPLETE line. One example of the expected output ordering can be seen in <https://trac.webkit.org/browser/webkit/trunk/LayoutTests/fast/events/fire-mousedown-while-pressing-mouse-button-expected.txt> Reopening to attach new patch. Created attachment 346248 [details]
Adjust layout test
(In reply to Daniel Bates from comment #9) > One example of the expected output ordering can be seen in > <https://trac.webkit.org/browser/webkit/trunk/LayoutTests/fast/events/fire- > mousedown-while-pressing-mouse-button-expected.txt> This example lacks a "PASS successfullyParsed is true" line. I used a more recent change as reference (r233387). Created attachment 346250 [details]
Adjust layout test
Comment on attachment 346250 [details] Adjust layout test View in context: https://bugs.webkit.org/attachment.cgi?id=346250&action=review Thank you Wenson for fixing the test. > LayoutTests/fast/forms/ios/click-should-not-suppress-misspelling.html:25 > +description(`To manually test, tap the input field to bring up the keyboard, and then tap on a part of the word in Please start this description with the purpose of the test. For example, in <https://trac.webkit.org/browser/webkit/trunk/LayoutTests/fast/events/fire-mousedown-while-pressing-mouse-button-expected.txt> the description starts with "This test verifies that we fire a mousedown event whenever pressing and holding a mouse button...". Minor: No need to use grave quotes as we do need any string interpolation. Double quotes are sufficient. Created attachment 346258 [details]
Patch for landing
Comment on attachment 346258 [details] Patch for landing Clearing flags on attachment: 346258 Committed r234449: <https://trac.webkit.org/changeset/234449> |