Bug 245984 – REGRESSION(255063@main): [ iOS ] editing/selection/ios/do-not-hide-selection-in-visible-container.html is a constant timeout

• Component: New Bugs
• Status: RESOLVED FIXED

[Hercules Hjalmarsson]

editing/selection/ios/do-not-hide-selection-in-visible-container.html

Is a constant timeout on iOS ToT and since 255063@main.

HISTORY:
https://results.webkit.org/?suite=layout-tests&test=editing/selection/ios/do-not-hide-selection-in-visible-container.html

DIFF:
@@ -1,18 +1,9 @@
+FAIL: Timed out waiting for notifyDone to be called

+First field  Second field  Third field  Fourth field

 Verifies that native selection UI is not suppressed when focusing an input that is inside an empty container with `overflow: hidden`, but is positioned absolutely such that it is still visible. To manually verify, click on each of the four buttons to move focus into editable areas. After tapping the first and third buttons, you should not see a caret view; after tapping the second and fourth buttons, you should see a caret view.

 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".

-Waiting for caret to hide.
-PASS Caret was hidden.
-Waiting for caret to show.
-PASS Caret was shown.
-Waiting for caret to hide.
-PASS Caret was hidden.
-Waiting for caret to show.
-PASS Caret was shown.
-PASS successfullyParsed is true

-TEST COMPLETE
-

[Radar WebKit Bug Importer]

<rdar://problem/100736393>

[Hercules Hjalmarsson]

This issue can be bisected to 255063@main using command:

run-webkit-tests --iterations=2  --ios-simulator editing/selection/ios/do-not-hide-selection-in-visible-container.html

[EWS]

Test gardening commit 255102@main (506efe392471): <https://commits.webkit.org/255102@main>

Reviewed commits have been landed. Closing PR #4948 and removing active labels.

[alan]

Created attachment 463283 [details]
[fast-cq]Patch

[Alexey Proskuryakov]

Looks like this progressed in 255598@main

[alan]

(In reply to Alexey Proskuryakov from comment #5)
> Looks like this progressed in 255598@main
Yeah, I did put that in the commit message.

[Alexey Proskuryakov]

Having it in the bug is useful too! I didn't notice it in the commit message though, so checked the history again.

[alan]

(In reply to Alexey Proskuryakov from comment #7)
> Having it in the bug is useful too! I didn't notice it in the commit message
> though, so checked the history again.
Indeed! I should have pre-fixed the bug (which I usually do, not sure why I didn't do it this time). Thank you.

[alan]

(In reply to zalan from comment #8)
> (In reply to Alexey Proskuryakov from comment #7)
> > Having it in the bug is useful too! I didn't notice it in the commit message
> > though, so checked the history again.
> Indeed! I should have pre-fixed the bug (which I usually do, not sure why I
> didn't do it this time). Thank you.
oops, not pre-fix, just include in the bug.

[EWS]

Committed 256083@main (9b6e03ce2f66): <https://commits.webkit.org/256083@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 463283 [details].