Bug 110379

Summary: Zoomed, slow-scrolling pages keep recreating tiles when scrolled
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: New BugsAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, simon.fraser, thorton, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch thorton: review+

Description Simon Fraser (smfr) 2013-02-20 14:05:15 PST
Zoomed, slow-scrolling pages keep recreating tiles when scrolled
Comment 1 Simon Fraser (smfr) 2013-02-20 14:13:13 PST
Created attachment 189374 [details]
Patch
Comment 2 Simon Fraser (smfr) 2013-02-20 14:13:30 PST
<rdar://problem/12840970>
Comment 3 Tim Horton 2013-02-20 14:16:34 PST
Comment on attachment 189374 [details]
Patch

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

> Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp:1624
> +    fprintf(stderr, "tileArea is %f x %f\n", tileArea.width(), tileArea.height());

Nah.

> LayoutTests/ChangeLog:10
> +        This test now just has one tile, rather than four. The apparent issue
> +        with the integral tile cache coverage rect being smaller than the visible rect
> +        is a rounding issue with the tile cache coverage dumping.

This sentence reads strangely.
Comment 4 Tim Horton 2013-02-20 14:26:21 PST
(In reply to comment #3)
> (From update of attachment 189374 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=189374&action=review
> 
> > Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp:1624
> > +    fprintf(stderr, "tileArea is %f x %f\n", tileArea.width(), tileArea.height());
> 
> Nah.
> 
> > LayoutTests/ChangeLog:10
> > +        This test now just has one tile, rather than four. The apparent issue
> > +        with the integral tile cache coverage rect being smaller than the visible rect
> > +        is a rounding issue with the tile cache coverage dumping.
> 
> This sentence reads strangely.

Nevermind, I get it on re-reading.
Comment 5 Simon Fraser (smfr) 2013-02-20 16:51:40 PST
https://trac.webkit.org/r143537