Bug 78976 - [mac] BackingStore::scroll() unnecessarily copies pixels around
Summary: [mac] BackingStore::scroll() unnecessarily copies pixels around
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac Unspecified
: P2 Normal
Assignee: mitz
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2012-02-18 14:49 PST by mitz
Modified: 2012-02-18 19:54 PST (History)
1 user (show)

See Also:


Attachments
Maintain a mapping from backing store coordinates to client coordinates in the most recently scrolled rect (10.13 KB, patch)
2012-02-18 15:04 PST, mitz
no flags Details | Formatted Diff | Diff
Maintain a mapping from backing store coordinates to client coordinates in the most recently scrolled rect (10.18 KB, patch)
2012-02-18 16:29 PST, mitz
andersca: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 2012-02-18 14:49:14 PST
BackingStore::scroll() unnecessarily copies pixels around
Comment 1 mitz 2012-02-18 14:50:43 PST
<rdar://problem/10891801>
Comment 2 mitz 2012-02-18 15:04:39 PST
Created attachment 127716 [details]
Maintain a mapping from backing store coordinates to client coordinates in the most recently scrolled rect
Comment 3 mitz 2012-02-18 16:29:28 PST
Created attachment 127720 [details]
Maintain a mapping from backing store coordinates to client coordinates in the most recently scrolled rect
Comment 4 Anders Carlsson 2012-02-18 19:47:08 PST
Comment on attachment 127720 [details]
Maintain a mapping from backing store coordinates to client coordinates in the most recently scrolled rect

View in context: https://bugs.webkit.org/attachment.cgi?id=127720&action=review

> Source/WebKit2/UIProcess/BackingStore.h:99
> +    // Contents of m_scrolledRect are offset by this amount (and wrapped around) with respect to

Please add an extra newline before this comment.
Comment 5 mitz 2012-02-18 19:54:24 PST
Fixed in <http://trac.webkit.org/r108181>.