This results in some functions (scrollBy) using ScrollView's implementation, and others (scrollTo) using FrameView's, and leads to a lot of confusion/brokenness.
Created attachment 166536 [details] patch
Thanks! http://trac.webkit.org/changeset/130123
Comment on attachment 166536 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=166536&action=review > Source/WebCore/platform/ScrollView.h:187 > + virtual void setScrollPosition(const IntPoint&); I recommend using the OVERRIDE macro in cases like this.
(In reply to comment #3) > (From update of attachment 166536 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=166536&action=review > > > Source/WebCore/platform/ScrollView.h:187 > > + virtual void setScrollPosition(const IntPoint&); > > I recommend using the OVERRIDE macro in cases like this. Hi, Darin! I did, on FrameView (the overriding subclass). I don't think it belongs on ScrollView since ScrollView is originating setScrollPosition, right?