RESOLVED FIXED 21314
Make scrollBackingStore cross-platform.
https://bugs.webkit.org/show_bug.cgi?id=21314
Summary Make scrollBackingStore cross-platform.
Dave Hyatt
Reported 2008-10-02 12:57:41 PDT
Make scrollBackingStore cross-platform.
Attachments
Patch (38.46 KB, patch)
2008-10-02 12:58 PDT, Dave Hyatt
sam: review+
Dave Hyatt
Comment 1 2008-10-02 12:58:37 PDT
Sam Weinig
Comment 2 2008-10-02 13:10:30 PDT
Comment on attachment 24028 [details] Patch + virtual void scroll(const IntSize& scrollDelta, const IntRect& rectToScroll, const IntRect& clipRect) { } There should not be the "= false" + if (canBlitOnScroll()) // The main frame can just blit the WebView window + // FIXME: Find a way to blit subframes without blitting overlapping content + hostWindow()->scroll(-scrollDelta, scrollViewRect, clipRect); + else + // We need to go ahead and repaint the entire backing store. Do it now before moving the + // plugins. + hostWindow()->repaint(updateRect, true, false, true); // Invalidate the backing store and repaint it synchronously This if-else needs braces as the statements + comments are more than 1 line. r=me
Dave Hyatt
Comment 3 2008-10-02 13:17:25 PDT
Fixed in r37204.
Note You need to log in before you can comment on or make changes to this bug.