RESOLVED FIXED 126362
Create a ThreadedScrollingTree subclass of ScrollingTree, and push all knowledge of the scrolling thread into it 
https://bugs.webkit.org/show_bug.cgi?id=126362
Summary Create a ThreadedScrollingTree subclass of ScrollingTree, and push all knowle...
Simon Fraser (smfr)
Reported 2014-01-01 12:37:51 PST
Create a ThreadedScrollingTree subclass of ScrollingTree, and push all knowledge of the scrolling thread into it
Attachments
Patch (34.79 KB, patch)
2014-01-01 12:43 PST, Simon Fraser (smfr) 		sam: review+
buildbot: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1 2014-01-01 12:43:16 PST
Created attachment 220190 [details] Patch
Sam Weinig
Comment 2 2014-01-01 12:57:35 PST
Comment on attachment 220190 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=220190&action=review > Source/WebCore/page/scrolling/ThreadedScrollingTree.h:64 > +#if PLATFORM(MAC) Can this be something more specific that PLATFORM(MAC), maybe ENABLE(WHEEL_EVENT_PHASE) or something?
Build Bot
Comment 3 2014-01-01 13:36:29 PST
Comment on attachment 220190 [details] Patch Attachment 220190 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/5107353917587456
Simon Fraser (smfr)
Comment 4 2014-01-01 15:13:14 PST
(In reply to comment #2) > (From update of attachment 220190 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=220190&action=review > > > Source/WebCore/page/scrolling/ThreadedScrollingTree.h:64 > > +#if PLATFORM(MAC) > > Can this be something more specific that PLATFORM(MAC), maybe ENABLE(WHEEL_EVENT_PHASE) or something? Yes, but not right now. Shame this wasn't done from the start :(
Simon Fraser (smfr)
Comment 5 2014-01-01 15:17:16 PST
https://trac.webkit.org/r161198
Note You need to log in before you can comment on or make changes to this bug.