Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(182)

Unified Diff: cc/occlusion_tracker.cc

Issue 11644008: Migrate from MathUtil::inverse() to gfx::Transform::GetInverse() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Patch for landing Created 7 years, 12 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/math_util_unittest.cc ('k') | cc/test/geometry_test_utils.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/occlusion_tracker.cc
diff --git a/cc/occlusion_tracker.cc b/cc/occlusion_tracker.cc
index e71c678591e2a8306ea70edd3256114866af3b06..c053fdac06ea2acc2372424c708c0060e3bfec2f 100644
--- a/cc/occlusion_tracker.cc
+++ b/cc/occlusion_tracker.cc
@@ -54,7 +54,7 @@ void OcclusionTrackerBase<LayerType, RenderSurfaceType>::leaveLayer(const LayerI
template<typename RenderSurfaceType>
static gfx::Rect screenSpaceClipRectInTargetSurface(const RenderSurfaceType* targetSurface, gfx::Rect screenSpaceClipRect)
{
- gfx::Transform inverseScreenSpaceTransform;
+ gfx::Transform inverseScreenSpaceTransform(gfx::Transform::kSkipInitialization);
if (!targetSurface->screenSpaceTransform().GetInverse(&inverseScreenSpaceTransform))
return targetSurface->contentRect();
@@ -124,7 +124,7 @@ void OcclusionTrackerBase<LayerType, RenderSurfaceType>::enterRenderTarget(const
bool enteringSubtreeThatMovesPixels = newAncestorThatMovesPixels && newAncestorThatMovesPixels != oldAncestorThatMovesPixels;
bool haveTransformFromScreenToNewTarget = false;
- gfx::Transform inverseNewTargetScreenSpaceTransform;
+ gfx::Transform inverseNewTargetScreenSpaceTransform(gfx::Transform::kSkipInitialization); // Note carefully, not used if screen space transform is uninvertible.
if (surfaceTransformsToScreenKnown(newTarget->renderSurface()))
haveTransformFromScreenToNewTarget = newTarget->renderSurface()->screenSpaceTransform().GetInverse(&inverseNewTargetScreenSpaceTransform);
@@ -353,7 +353,7 @@ bool OcclusionTrackerBase<LayerType, RenderSurfaceType>::occluded(const LayerTyp
DCHECK(renderTarget->renderSurface());
DCHECK(renderTarget == m_stack.back().target);
- gfx::Transform inverseDrawTransform;
+ gfx::Transform inverseDrawTransform(gfx::Transform::kSkipInitialization);
if (!drawTransform.GetInverse(&inverseDrawTransform))
return false;
@@ -396,7 +396,7 @@ gfx::Rect OcclusionTrackerBase<LayerType, RenderSurfaceType>::unoccludedContentR
DCHECK(renderTarget->renderSurface());
DCHECK(renderTarget == m_stack.back().target);
- gfx::Transform inverseDrawTransform;
+ gfx::Transform inverseDrawTransform(gfx::Transform::kSkipInitialization);
if (!drawTransform.GetInverse(&inverseDrawTransform))
return contentRect;
@@ -445,7 +445,7 @@ gfx::Rect OcclusionTrackerBase<LayerType, RenderSurfaceType>::unoccludedContribu
return contentRect;
gfx::Transform drawTransform = forReplica ? surface->replicaDrawTransform() : surface->drawTransform();
- gfx::Transform inverseDrawTransform;
+ gfx::Transform inverseDrawTransform(gfx::Transform::kSkipInitialization);
if (!drawTransform.GetInverse(&inverseDrawTransform))
return contentRect;
« no previous file with comments | « cc/math_util_unittest.cc ('k') | cc/test/geometry_test_utils.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698