RESOLVED FIXED 133022
Split scrolling tree ScrollingNodes into FrameScrollingNodes and OverflowScrollingNodes 
https://bugs.webkit.org/show_bug.cgi?id=133022
Summary Split scrolling tree ScrollingNodes into FrameScrollingNodes and OverflowScro...
Simon Fraser (smfr)
Reported 2014-05-16 18:42:07 PDT
Split scrolling tree ScrollingNodes into FrameScrollingNodes and OverflowScrollingNodes
Attachments
Patch (278.39 KB, patch)
2014-05-16 18:49 PDT, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
EWS for EFL (279.06 KB, patch)
2014-05-19 11:06 PDT, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1 2014-05-16 18:49:07 PDT
Created attachment 231612 [details] Patch
WebKit Commit Bot
Comment 2 2014-05-16 18:50:28 PDT
Attachment 231612 [details] did not pass style-queue: ERROR: Source/WebKit2/Shared/Scrolling/RemoteScrollingCoordinatorTransaction.cpp:167: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebKit2/Shared/Scrolling/RemoteScrollingCoordinatorTransaction.cpp:175: Multi line control clauses should use braces. [whitespace/braces] [4] Total errors found: 2 in 56 files If any of these errors are false positives, please file a bug against check-webkit-style.
Simon Fraser (smfr)
Comment 3 2014-05-19 11:06:25 PDT
Created attachment 231703 [details] EWS for EFL
WebKit Commit Bot
Comment 4 2014-05-19 11:09:40 PDT
Attachment 231703 [details] did not pass style-queue: ERROR: Source/WebKit2/Shared/Scrolling/RemoteScrollingCoordinatorTransaction.cpp:167: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebKit2/Shared/Scrolling/RemoteScrollingCoordinatorTransaction.cpp:175: Multi line control clauses should use braces. [whitespace/braces] [4] Total errors found: 2 in 57 files If any of these errors are false positives, please file a bug against check-webkit-style.
Simon Fraser (smfr)
Comment 5 2014-05-19 13:42:35 PDT
http://trac.webkit.org/changeset/169063
Andreas Kling
Comment 6 2014-05-19 14:04:53 PDT
Comment on attachment 231703 [details] EWS for EFL Removing from review queue since this already landed.
Note You need to log in before you can comment on or make changes to this bug.