WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Dave Hyatt
Comment 1
2008-10-02 12:58:37 PDT
Created
attachment 24028
[details]
Patch
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.
Top of Page
Format For Printing
XML
Clone This Bug