Bug 81858

Summary: Enable layout testing of the scroll animator.
Product: WebKit Reporter: Scott Byer <scottbyer>
Component: New BugsAssignee: Scott Byer <scottbyer>
Severity: Normal CC: abarth, dglazkov, fishd, jamesr, tkent, tony, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 81436    
Description Flags
Patch none

Description Scott Byer 2012-03-21 18:15:00 PDT
Enable layout testing of the scroll animator.
Comment 1 Scott Byer 2012-03-21 18:22:13 PDT
Created attachment 133159 [details]
Comment 2 WebKit Review Bot 2012-03-21 18:24:24 PDT
Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.
Comment 3 Scott Byer 2012-03-22 16:46:00 PDT
James, if you get a chance, could you take a look? Robert and I decided that crbug.com/118507 isn't critical for m19, so if you're slammed with other m19 things, this can wait.
Comment 4 James Robinson 2012-03-23 09:16:28 PDT
Comment on attachment 133159 [details]

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

Public WK API changes look fine. I'm not as familiar with how InternalSettings are supposed to work, so perhaps someone who's more familiar with that logic would be able to review. I don't think this is critical for M19 and I'm semi-unavailable.

> Tools/DumpRenderTree/chromium/WebViewHost.cpp:769
> +    postDelayedTask(new HostMethodTask(this, &WebViewHost::serviceAnimation), 0);

Hm, I'm a bit surprised that this didn't have any impact on the fast/animation/ tests.
Comment 5 James Robinson 2012-03-26 18:33:03 PDT
Comment on attachment 133159 [details]

R=me assuming it actually is a no-op (and doesn't break fast/animations)
Comment 6 WebKit Review Bot 2012-03-26 19:41:37 PDT
Comment on attachment 133159 [details]

Clearing flags on attachment: 133159

Committed r112196: <http://trac.webkit.org/changeset/112196>
Comment 7 WebKit Review Bot 2012-03-26 19:41:42 PDT
All reviewed patches have been landed.  Closing bug.
Comment 8 Tony Chang 2013-03-21 16:31:03 PDT
Did we ever write any layout tests that use internals.settings.setEnableScrollAnimator/scrollAnimatorEnabled?  I don't see any calls to these methods in LayoutTests and was wondering if I could remove these methods.