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

Unified Diff: Source/core/rendering/svg/SVGRenderingContext.cpp

Issue 59063003: Don't coerce pointers to compositor layer mappings to booleans. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Respond to reviewer feedback. Created 7 years, 1 month 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
Index: Source/core/rendering/svg/SVGRenderingContext.cpp
diff --git a/Source/core/rendering/svg/SVGRenderingContext.cpp b/Source/core/rendering/svg/SVGRenderingContext.cpp
index 2f79029eec24b5f88ec8d747d799f0068ee4d31e..4b735ae98d4c982ac106f1575b2ca73978fd44d8 100644
--- a/Source/core/rendering/svg/SVGRenderingContext.cpp
+++ b/Source/core/rendering/svg/SVGRenderingContext.cpp
@@ -228,7 +228,9 @@ void SVGRenderingContext::calculateTransformationToOutermostCoordinateSystem(con
// We can stop at compositing layers, to match the backing resolution.
// FIXME: should we be computing the transform to the nearest composited layer,
// or the nearest composited layer that does not paint into its ancestor?
- if (layer->compositedLayerMapping())
+ // I think this is the nearest composited ancestor since we will inherit its
+ // transforms in the composited layer tree.
+ if (layer->hasCompositedLayerMapping())
break;
layer = layer->parent();

Powered by Google App Engine
This is Rietveld 408576698