Can't scroll iframe after toggling it to display:none and back

[Simon Fraser (smfr)]

Can't scroll iframe after toggling it to display:none and back

[Simon Fraser (smfr)]

Created attachment 331442 [details]
Patch

[Simon Fraser (smfr)]

rdar://problem/13234778

[WebKit Commit Bot]

Comment on attachment 331442 [details]
Patch

Clearing flags on attachment: 331442

Committed r227011: <https://trac.webkit.org/changeset/227011>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Ryan Haddad]

This change caused LayoutTest fast/frames/hidpi-position-iframe-on-device-pixel.html to consistently time out macOS Debug WK2 bots:

https://build.webkit.org/results/Apple%20High%20Sierra%20Debug%20WK2%20(Tests)/r227174%20(1674)/results.html

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=fast%2Fframes%2Fhidpi-position-iframe-on-device-pixel.html

[Simon Fraser (smfr)]

Thanks, will look.

[Simon Fraser (smfr)]

I can't reproduce the timeouts.

[Simon Fraser (smfr)]

The test is pretty slow, though. Maybe I made it slower.

[Simon Fraser (smfr)]

Yeah, now we're adding to the scrollableAreaSet even before we know that the frame is scrollable.

[Simon Fraser (smfr)]

Fixing via bug 181959.