Bug 237069 – REGRESSION (288925?): [iOS] TestWebKitAPI.RequestTextInputContext.TextInteraction_FocusingReadOnlyElementShouldScrollToReveal is failing

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Ryan Haddad]

TestWebKitAPI.RequestTextInputContext.TextInteraction_FocusingReadOnlyElementShouldScrollToReveal has been pretty consistently failing on iOS bots since r288941. The regression appears to be somewhere in this range: https://github.com/WebKit/WebKit/compare/69ae9efe43bc4931906b774dcd44265ab2f542bb...505f6ed4699de698ff288413c7a61cb4e07ba9fa


   TestWebKitAPI.RequestTextInputContext.TextInteraction_FocusingReadOnlyElementShouldScrollToReveal
        2022-02-22 13:54:40.416 TestWebKitAPI[70971:37875135] Writing analzed variants.
        2022-02-22 13:54:40.432 TestWebKitAPI[70971:37875135] Writing analzed variants.
        LEAK: 1 WebProcessPool
        LEAK: 1 WebPageProxy
        
        /Volumes/Data/worker/ios-simulator-15-debug/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/RequestTextInputContext.mm:667
        Value of: didScroll
          Actual: false
        Expected: true

https://results.webkit.org/?suite=api-tests&test=TestWebKitAPI.RequestTextInputContext.TextInteraction_FocusingReadOnlyElementShouldScrollToReveal

[Radar WebKit Bug Importer]

<rdar://problem/89325305>

[Wenson Hsieh]

At a glance, nothing in that range sticks out to me as the culprit. I'll take a closer look...

[Wenson Hsieh]

Created attachment 452925 [details]
Repro?

[Wenson Hsieh]

(In reply to Wenson Hsieh from comment #3)
> Created attachment 452925 [details]
> Repro?

I wasn't able to reproduce this failure locally; going to see if EWS can catch it at least once in 200 runs.

[Wenson Hsieh]

(In reply to Wenson Hsieh from comment #4)
> (In reply to Wenson Hsieh from comment #3)
> > Created attachment 452925 [details]
> > Repro?
> 
> I wasn't able to reproduce this failure locally; going to see if EWS can
> catch it at least once in 200 runs.

Looking at https://ews-build.webkit.org/#/builders/9/builds/65529/steps/14/logs/stdio, it seems all 200 of the TextInteraction_FocusingReadOnlyElementShouldScrollToReveal_* tests passed.

[Wenson Hsieh]

Created attachment 453000 [details]
Speculative fix

[Wenson Hsieh]

Comment on attachment 453000 [details]
Speculative fix

Thanks for the review!

[EWS]

Committed r290389 (247704@main): <https://commits.webkit.org/247704@main>

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