[webkit-changes] [WebKit/WebKit] ef3f34: editing/selection/ios/hide-selection-in-hidden-con...
Wenson Hsieh
noreply at github.com
Wed May 10 16:18:11 PDT 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: ef3f34692b73ffde48adf3a63a2d64b204929402
https://github.com/WebKit/WebKit/commit/ef3f34692b73ffde48adf3a63a2d64b204929402
Author: Wenson Hsieh <wenson_hsieh at apple.com>
Date: 2023-05-10 (Wed, 10 May 2023)
Changed paths:
M LayoutTests/editing/selection/ios/hide-selection-in-hidden-contenteditable-expected.txt
M LayoutTests/editing/selection/ios/hide-selection-in-hidden-contenteditable-frame-expected.txt
M LayoutTests/editing/selection/ios/hide-selection-in-hidden-contenteditable-frame.html
M LayoutTests/editing/selection/ios/hide-selection-in-hidden-contenteditable.html
Log Message:
-----------
editing/selection/ios/hide-selection-in-hidden-contenteditable.html times out on some test runners
https://bugs.webkit.org/show_bug.cgi?id=256608
rdar://107725234
Reviewed by Ryosuke Niwa.
This patch contains a couple of speculative fixes for this failing test, along with some logging to
help diagnose the test failure in case the speculative fixes don't work. I suspect that this is due
to the prior test, `editing/selection/ios/hide-selection-in-hidden-contenteditable-frame.html`,
ending too early (just as the keyboard is about to appear). Address this by cleaning up this
previous test so that it waits for the keyboard to appear and then disappear before ending, by
adding a missing `await` in the `waitForSelectionChangeInSubframe` helper function.
We also sprinkle some `console.log` messages throughout both tests, so that even if the test times
out, we'll be able to observe how far along it got.
* LayoutTests/editing/selection/ios/hide-selection-in-hidden-contenteditable-expected.txt:
* LayoutTests/editing/selection/ios/hide-selection-in-hidden-contenteditable-frame-expected.txt:
* LayoutTests/editing/selection/ios/hide-selection-in-hidden-contenteditable-frame.html:
* LayoutTests/editing/selection/ios/hide-selection-in-hidden-contenteditable.html:
Canonical link: https://commits.webkit.org/263936@main
More information about the webkit-changes
mailing list