Bug 76541

Summary: WK2+AC+Tiling: Project the visible rect on the transformed layer instead of mapping it.
Product: WebKit Reporter: Jocelyn Turcotte <jturcotte>
Component: New BugsAssignee: Jocelyn Turcotte <jturcotte>
Status: RESOLVED INVALID    
Severity: Normal CC: noam
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Jocelyn Turcotte
Reported 2012-01-18 07:22:01 PST
WK2+AC+Tiling: Project the visible rect on the transformed layer instead of mapping it.
Attachments
Patch (2.64 KB, patch)
2012-01-18 07:22 PST, Jocelyn Turcotte 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Jocelyn Turcotte
Comment 1 2012-01-18 07:22:37 PST
Created attachment 122926 [details] Patch
Jocelyn Turcotte
Comment 2 2012-01-18 07:24:43 PST
This is the result of yesterday's IRC discussion. Modifying the TiledBackingStore to test each tile rect back to the visible/cover area wouldn't scale as well since we would have to test each single tile each iteration. I found out that clampedBoundsOfProjectedQuad is exactly what I was needing so I'm using it instead here.
Noam Rosenthal
Comment 3 2012-01-18 09:03:30 PST
Any reason why you don't just merge this into 74720?
Jocelyn Turcotte
Comment 4 2012-01-19 07:41:17 PST
This patch got merged pushed with the content of bug #74720. Got it split since this changes the behavior while the rest was just moving code around, but the other patch was huge already anyway.
Eric Seidel (no email)
Comment 5 2012-01-30 16:00:43 PST
Comment on attachment 122926 [details] Patch Cleared review? from attachment 122926 [details] so that this bug does not appear in http://webkit.org/pending-review. If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).
Note You need to log in before you can comment on or make changes to this bug.