[chromium] Consistent checking for clipped rects when we need the computed result...
authordanakj@chromium.org <danakj@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 17 Apr 2012 00:21:46 +0000 (00:21 +0000)
committerdanakj@chromium.org <danakj@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 17 Apr 2012 00:21:46 +0000 (00:21 +0000)
https://bugs.webkit.org/show_bug.cgi?id=83543

Reviewed by Adrienne Walker.

It should not be possible to make a rect in layer space that is clipped
by the camera but for which the screen space transform gives a
rectilinear output. But use consistent methods for checking that the
result remains enclosed within the actual pixels.

One day when clipped is true, we can find an interior axis-aligned rect
within the clipped result, and checking clipped explicitly makes this
more clear.

Covered by existing tests.

* platform/graphics/chromium/cc/CCOcclusionTracker.cpp:
(WebCore::computeOcclusionBehindLayer):
(WebCore::::markOccludedBehindLayer):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@114324 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog

index 28a431b..b8294c6 100644 (file)
@@ -1,3 +1,25 @@
+2012-04-16  Dana Jansens  <danakj@chromium.org>
+
+        [chromium] Consistent checking for clipped rects when we need the computed result enclosed within the real result
+        https://bugs.webkit.org/show_bug.cgi?id=83543
+
+        Reviewed by Adrienne Walker.
+
+        It should not be possible to make a rect in layer space that is clipped
+        by the camera but for which the screen space transform gives a
+        rectilinear output. But use consistent methods for checking that the
+        result remains enclosed within the actual pixels.
+
+        One day when clipped is true, we can find an interior axis-aligned rect
+        within the clipped result, and checking clipped explicitly makes this
+        more clear.
+
+        Covered by existing tests.
+
+        * platform/graphics/chromium/cc/CCOcclusionTracker.cpp:
+        (WebCore::computeOcclusionBehindLayer):
+        (WebCore::::markOccludedBehindLayer):
+
 2012-04-16  Erik Arvidsson  <arv@chromium.org>
 
         [V8] Don't delete the per context data until the V8IsolatedContext is deleted