[Webkit-unassigned] [Bug 87167] [chromium] Cleanup scissor rect computation/use with damage

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=87167





--- Comment #11 from zlieber at chromium.org  2012-05-30 10:04:41 PST ---
(From update of attachment 143848)
View in context: https://bugs.webkit.org/attachment.cgi?id=143848&action=review

Just one detail below, the rest is implemented.

>> Source/WebCore/platform/graphics/chromium/cc/CCRenderSurface.cpp:474
>> +        surfaceDamageRect = inverseScreenSpaceTransform.mapRect(damageRect);
> 
> This should be using CCMathUtil::projectClippedRect. And the hasPerspective check can just go away. Does that make it more reasonable to do this externally, as it's just a project on the root rect using the surface's screen space inverse transform?
> 
> Shawn can you verify what I just said here is true?

It sure would if we could remove the call to this from CCLayerTreeHostImpl::drawLayers and just use root surface's scissorRect instead. Would that make sense?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.