Bug 164905 – Make it possible to test non-stable-state scrolling on iOS

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

[Simon Fraser (smfr)]

I need to be able to test scrolling behavior (tile coverage, scrolling tree state) when the viewport is not in a stable state, to simulate what happens in the middle of scrolling or zooming.

[Simon Fraser (smfr)]

I have a patch but the associated layer tree dump, with visible and coverage rects, is flakey.

[Simon Fraser (smfr)]

The source of the flakiness is that sometimes TileController::adjustTileCoverageRect() has CoverageForVisibleArea and sometimes CoverageForVerticalScrolling. That's affected by FrameView::speculativeTilingEnabled() which is in turn affected by FrameView::wasScrolledByUser(). FrameView:: wasScrolledByUser() is being set via EventHandler::sendScrollEvent() (why?). This is called from FrameView::scrollPositionChanged(), which uses the eventThrottlingDelay() from WebKit2.

There are also timers at various points in this chain. It's all very hard to get stable for testing.

[Simon Fraser (smfr)]

In my particular case m_estimatedLatency is 16ms so we decide to send a scroll event, setting FrameView's scrolledByUser, which then triggers additional tile coverage.

[Simon Fraser (smfr)]

Created attachment 295133 [details]
Patch

[Simon Fraser (smfr)]

Created attachment 295135 [details]
Patch

[Simon Fraser (smfr)]

Need to redo parts of this patch after bug 164980.

[Simon Fraser (smfr)]

Created attachment 295264 [details]
Patch

Comment on attachment 295264 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=295264&action=review

> Source/WebKit2/ChangeLog:11
> +        scrolling and layer trees in these transients states.

“transients”?

> Source/WebKit2/UIProcess/API/Cocoa/WKWebViewPrivate.h:293
> +@property (nonatomic, setter=_setStableStateOverride:) NSNumber *_stableStateOverride WK_API_AVAILABLE(ios(WK_IOS_TBA));

Can be readonly in this class, with subclasses redeclaring as readwrite.

> Tools/WebKitTestRunner/cocoa/TestRunnerWKWebView.mm:50
> +@property (nonatomic, copy) NSNumber *testRunnerStableStateOverride;

Strange to add this level of indirection. Can just declare a RetainPtr<NSNumber> ivar to back the existing property.

> Tools/WebKitTestRunner/ios/UIScriptControllerIOS.mm:63
> +    [webView _doAfterNextPresentationUpdate:^ {

Extra space after the ^, and I thought we preferred C++ lambdas with explicit capture over Objective-C blocks.

> Tools/WebKitTestRunner/ios/UIScriptControllerIOS.mm:423
> +    if (overrideValue)
> +        webView._stableStateOverride = @(overrideValue.value());
> +    else
> +        webView._stableStateOverride = nil;

Can be written more succinctly with a ternary operator.

[Simon Fraser (smfr)]

https://trac.webkit.org/changeset/208926

[Csaba Osztrogonác]

(In reply to comment #9)
> https://trac.webkit.org/changeset/208926

It broke the Apple Mac build on all buildbots, see build.webkit.org for details.

[Simon Fraser (smfr)]

Fixing.