Bug 42338

Summary: Avoid creating huge compositing layers for elements that project outside the viewport
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: Layout and RenderingAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, eric, mitz, webkit.review.bot
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: OS X 10.5   
Attachments:
Description Flags
Patch
none
Patch mitz: review+

Simon Fraser (smfr)
Reported 2010-07-14 22:51:02 PDT
When a page has an abs. pos. div at -1000px, -1000px, or uses text-indent: -9999px, then we should avoid making huge, tiled compositing layers to try to encompass these elements. More practically, this means constraining compositing layer size to the size of the enclosing document, or the enclosing overflow div.
Attachments
Patch (35.45 KB, patch)
2010-07-14 23:01 PDT, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
Patch (49.85 KB, patch)
2010-07-15 12:27 PDT, Simon Fraser (smfr) 		mitz: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1 2010-07-14 22:51:31 PDT
<rdar://problem/8187624>
Simon Fraser (smfr)
Comment 2 2010-07-14 23:01:07 PDT
Created attachment 61609 [details] Patch
Simon Fraser (smfr)
Comment 3 2010-07-14 23:02:22 PDT
Comment on attachment 61609 [details] Patch Not yet ready for review. enclosingOverflowScrollAncestor() should look for any overflow/clipping, not just those that result in scrolling.
Simon Fraser (smfr)
Comment 4 2010-07-14 23:04:05 PDT
Comment on attachment 61609 [details] Patch s/convertContainerToLocalQuadI/convertContainerToLocalQuad
Simon Fraser (smfr)
Comment 5 2010-07-15 12:27:46 PDT
Created attachment 61701 [details] Patch
Simon Fraser (smfr)
Comment 6 2010-07-15 13:09:58 PDT
http://trac.webkit.org/changeset/63452
WebKit Review Bot
Comment 7 2010-07-15 13:43:26 PDT
http://trac.webkit.org/changeset/63452 might have broken SnowLeopard Intel Release (Tests)
Note You need to log in before you can comment on or make changes to this bug.