Bug 102908

Summary: Make TiledBackingStore::adjustForContentsRect() handle the case when a bound rect does not contain a cover rect.
Product: WebKit Reporter: Dongseong Hwang <dongseong.hwang>
Component: Layout and RenderingAssignee: Dongseong Hwang <dongseong.hwang>
Status: NEW    
Severity: Normal CC: kenneth
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 102891    
Attachments:
Description Flags
Patch
none
Patch none

Dongseong Hwang
Reported 2012-11-21 02:29:06 PST
adjustForContentsRect() expects a bound rect to cover a cover rect. However, when TiledBackingStore is used by CoordinatedGraphicsLayer, this expectation is wrong, which r134048 clarified. adjustForContentsRect() does not yet handle the case when one side of the cover rect is longer than the bound rect. This patch makes the method handle the case.
Attachments
Patch (2.20 KB, patch)
2012-11-21 02:34 PST, Dongseong Hwang 		no flags
DetailsFormatted DiffDiff
Patch (2.78 KB, patch)
2012-11-21 02:42 PST, Dongseong Hwang 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Dongseong Hwang
Comment 1 2012-11-21 02:34:08 PST
Created attachment 175387 [details] Patch
Dongseong Hwang
Comment 2 2012-11-21 02:34:46 PST
r134048 is Bug 101656. It is a follow-up patch of r134048.
Dongseong Hwang
Comment 3 2012-11-21 02:42:18 PST
Created attachment 175389 [details] Patch
Anders Carlsson
Comment 4 2014-02-05 11:05:21 PST
Comment on attachment 175389 [details] Patch Clearing review flag on patches from before 2014. If this patch is still relevant, please reset the r? flag.
Note You need to log in before you can comment on or make changes to this bug.