RESOLVED FIXED147963
Move RenderBox-specific Scroll Snap code from RenderElement to RenderBox 
https://bugs.webkit.org/show_bug.cgi?id=147963
Summary Move RenderBox-specific Scroll Snap code from RenderElement to RenderBox
Brent Fulgham
Reported 2015-08-12 17:42:16 PDT
Certain scroll snap routines were added to RenderElement before we decided that this code only applied to RenderBoxes. Rather than adding type checking in RenderElement to handle these objects, only do the RenderBox-specific logic in the RenderBox class.
Attachments
Patch (4.29 KB, patch)
2015-08-12 17:49 PDT, Brent Fulgham 		simon.fraser: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Brent Fulgham
Comment 1 2015-08-12 17:49:24 PDT
Created attachment 258863 [details] Patch
Brent Fulgham
Comment 2 2015-08-12 18:20:44 PDT
Committed r188370: <http://trac.webkit.org/changeset/188370>
alan
Comment 3 2015-08-12 18:32:32 PDT
Comment on attachment 258863 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=258863&action=review > Source/WebCore/rendering/RenderBox.h:638 > + void willBeRemovedFromTree() override; I'd write out 'virtual' here.
Brent Fulgham
Comment 4 2015-08-12 19:20:20 PDT
(In reply to comment #3) > Comment on attachment 258863 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=258863&action=review > > > Source/WebCore/rendering/RenderBox.h:638 > > + void willBeRemovedFromTree() override; > > I'd write out 'virtual' here. Darin says we don't do that anymore. It's redundant with the 'override' declaration.
alan
Comment 5 2015-08-12 19:22:55 PDT
Comment on attachment 258863 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=258863&action=review >>> Source/WebCore/rendering/RenderBox.h:638 >>> + void willBeRemovedFromTree() override; >> >> I'd write out 'virtual' here. > > Darin says we don't do that anymore. It's redundant with the 'override' declaration. Ok, thanks. Good to know.
Note You need to log in before you can comment on or make changes to this bug.