Bug 108719 - [chromium] Flings should not bubble up to enclosing scrollables when main-thread touch scrolling
Summary: [chromium] Flings should not bubble up to enclosing scrollables when main-thr...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Terry Anderson
URL:
Keywords:
Depends on: 108849
Blocks:
  Show dependency treegraph
 
Reported: 2013-02-01 15:53 PST by Terry Anderson
Modified: 2013-02-07 16:05 PST (History)
13 users (show)

See Also:


Attachments
WIP (42.71 KB, patch)
2013-02-01 15:56 PST, Terry Anderson
no flags Details | Formatted Diff | Diff
Patch (1.80 KB, patch)
2013-02-04 13:12 PST, Terry Anderson
no flags Details | Formatted Diff | Diff
Patch (1.82 KB, patch)
2013-02-06 13:45 PST, Terry Anderson
no flags Details | Formatted Diff | Diff
Patch (1.91 KB, patch)
2013-02-06 17:09 PST, Terry Anderson
no flags Details | Formatted Diff | Diff
Patch (1.79 KB, patch)
2013-02-07 06:45 PST, Terry Anderson
no flags Details | Formatted Diff | Diff
Patch (1.83 KB, patch)
2013-02-07 14:04 PST, Terry Anderson
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Terry Anderson 2013-02-01 15:53:30 PST
Flings should not bubble up to enclosing scrollables when main-thread touch scrolling.
Comment 1 Terry Anderson 2013-02-01 15:56:20 PST
Created attachment 186177 [details]
WIP
Comment 2 WebKit Review Bot 2013-02-01 16:01:06 PST
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 Robert Kroeger 2013-02-04 12:03:08 PST
FWIW: the fling code in WebViewImpl and related event changes look good to me.
Comment 4 Terry Anderson 2013-02-04 13:12:29 PST
Created attachment 186445 [details]
Patch
Comment 5 WebKit Review Bot 2013-02-04 13:21:30 PST
Comment on attachment 186445 [details]
Patch

Attachment 186445 [details] did not pass cr-linux-debug-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/16374325
Comment 6 WebKit Review Bot 2013-02-04 14:07:08 PST
Comment on attachment 186445 [details]
Patch

Attachment 186445 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/16371423
Comment 7 Peter Beverloo (cr-android ews) 2013-02-04 14:28:13 PST
Comment on attachment 186445 [details]
Patch

Attachment 186445 [details] did not pass cr-android-ews (chromium-android):
Output: http://queues.webkit.org/results/16378284
Comment 8 Terry Anderson 2013-02-06 13:45:46 PST
Created attachment 186912 [details]
Patch
Comment 9 Peter Beverloo (cr-android ews) 2013-02-06 14:15:31 PST
Comment on attachment 186912 [details]
Patch

Attachment 186912 [details] did not pass cr-android-ews (chromium-android):
Output: http://queues.webkit.org/results/16394390
Comment 10 Terry Anderson 2013-02-06 17:09:19 PST
Created attachment 186953 [details]
Patch
Comment 11 Terry Anderson 2013-02-07 06:45:36 PST
Created attachment 187091 [details]
Patch
Comment 12 Eric Seidel (no email) 2013-02-07 13:10:32 PST
Looks very reasonable. :)  How does one test this?
Comment 13 Build Bot 2013-02-07 13:33:36 PST
Comment on attachment 187091 [details]
Patch

Attachment 187091 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/16428331
Comment 14 Terry Anderson 2013-02-07 13:57:22 PST
(In reply to comment #12)
> Looks very reasonable. :)  How does one test this?

I have tested the behavior of the new event type GestureScrollUpdateWithoutPropagation in isolation for https://bugs.webkit.org/show_bug.cgi?id=108849, and testing fling itself (such as in fast/events/touch/gesture/pad-gesture-fling.html) is notoriously flaky.
Comment 15 Terry Anderson 2013-02-07 14:04:04 PST
Created attachment 187173 [details]
Patch
Comment 16 Eric Seidel (no email) 2013-02-07 14:32:14 PST
Comment on attachment 187173 [details]
Patch

I worry w/o automated testing this will break again. :(
Comment 17 Terry Anderson 2013-02-07 14:48:11 PST
(In reply to comment #16)
> (From update of attachment 187173 [details])
> I worry w/o automated testing this will break again. :(

I definitely agree that some reliable automated testing for fling gestures would be welcome (but in speaking with @rjkroege, this will likely require a substantial effort to make robust). I have filed https://bugs.webkit.org/show_bug.cgi?id=109234 to track.
Comment 18 WebKit Review Bot 2013-02-07 16:05:22 PST
Comment on attachment 187173 [details]
Patch

Clearing flags on attachment: 187173

Committed r142197: <http://trac.webkit.org/changeset/142197>
Comment 19 WebKit Review Bot 2013-02-07 16:05:29 PST
All reviewed patches have been landed.  Closing bug.