[webkit-changes] [WebKit/WebKit] f2815c: [iOS] A couple of inline prediction layout tests a...

Wenson Hsieh noreply at github.com
Wed Jan 17 15:12:34 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: f2815ca5349748843abc9a3c2b3b675b359df489
      https://github.com/WebKit/WebKit/commit/f2815ca5349748843abc9a3c2b3b675b359df489
  Author: Wenson Hsieh <wenson_hsieh at apple.com>
  Date:   2024-01-17 (Wed, 17 Jan 2024)

  Changed paths:
    M LayoutTests/editing/input/ios/typing-with-inline-predictions-expected.txt
    M LayoutTests/editing/input/ios/typing-with-inline-predictions.html
    M LayoutTests/editing/selection/ios/tap-to-change-selection-after-accepting-inline-prediction.html

  Log Message:
  -----------
  [iOS] A couple of inline prediction layout tests are flaky
https://bugs.webkit.org/show_bug.cgi?id=267639
rdar://121118748

Reviewed by Megan Gardner.

Attempt to triage a couple of flaky layout tests that exercise inline text predictions. I wasn't
able to reproduce any test failures when running either of these (running them fully parallel for
several hundred iterations, or when running them in order along with the rest of the test group for
several dozen iterations).

These are purely speculative adjustments to the flaky tests.

* LayoutTests/editing/input/ios/typing-with-inline-predictions-expected.txt:
* LayoutTests/editing/input/ios/typing-with-inline-predictions.html:

Instead of only waiting for `typeCharacter`, wait for the editing container to receive an `keyup`
event as well before trying to send the next set of key events.

Also, temporarily add a 100 ms delay between keystrokes. If this patch fixes the flakiness observed
in our automation, we should try removing this delay and see if these tests start flaking again. If
so, we likely have another source of flakiness when synthesizing key events, which the above change
to wait for the `keyup` event does not address.

Lastly, add a `console.log` here so that if the test begins to time out due to never observing an
`keyup` event, we can easily tell.

* LayoutTests/editing/selection/ios/tap-to-change-selection-after-accepting-inline-prediction.html:

Same as above, but additionally remove the `allowsInlinePredictions=true` test option. This test
doesn't actually need this option, because it explicitly sets inline predictions; I think it's
possible that, by enabling inline predictions, the real system predictions (or lack thereof)
override the custom prediction ("I want to c<elebrate>").

Canonical link: https://commits.webkit.org/273149@main




More information about the webkit-changes mailing list