Bug 171923 - Refactor ScrollingCoordinator::setSynchronousScrollingReasons to accept a FrameView
Summary: Refactor ScrollingCoordinator::setSynchronousScrollingReasons to accept a Fra...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Frames (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Frédéric Wang (:fredw)
URL:
Keywords:
Depends on:
Blocks: 171667 172806
  Show dependency tree / graph
 
Reported: 2017-05-10 08:09 PDT by Frédéric Wang (:fredw)
Modified: 2017-06-01 10:46 PDT (History)
7 users (show)

See Also:

Attachments
Patch (6.82 KB, patch)
2017-05-10 08:24 PDT, Frédéric Wang (:fredw) 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Frédéric Wang (:fredw) 2017-05-10 08:09:03 PDT 
Currently ScrollingCoordinator::setSynchronousScrollingReasons implementations only take SynchronousScrollingReasons and applies it to the main frame. We should pass the desired FrameView to which the reasons apply in order to prepare support for fast scrolling in other frames (bug 171667).
Comment 1 Frédéric Wang (:fredw) 2017-05-10 08:24:35 PDT 
Created attachment 309606 [details]
Patch
Comment 2 WebKit Commit Bot 2017-05-10 09:57:54 PDT 
Comment on attachment 309606 [details]
Patch

Clearing flags on attachment: 309606

Committed r216592: <http://trac.webkit.org/changeset/216592>
Comment 3 WebKit Commit Bot 2017-05-10 09:57:57 PDT 
All reviewed patches have been landed.  Closing bug.