 Chromium Code Reviews
 Chromium Code Reviews Issue 23785014:
  [SVG] Resources should be laid out in dependecy order.  (Closed) 
  Base URL: svn://svn.chromium.org/blink/trunk
    
  
    Issue 23785014:
  [SVG] Resources should be laid out in dependecy order.  (Closed) 
  Base URL: svn://svn.chromium.org/blink/trunk| Index: Source/core/rendering/RenderView.cpp | 
| diff --git a/Source/core/rendering/RenderView.cpp b/Source/core/rendering/RenderView.cpp | 
| index c18c9d014b5f3a0fa1b7c34fcda6587871744287..4505621444c8b0fa735adb38f469dd3fcebec7da 100644 | 
| --- a/Source/core/rendering/RenderView.cpp | 
| +++ b/Source/core/rendering/RenderView.cpp | 
| @@ -42,6 +42,7 @@ | 
| #include "core/rendering/RenderLayerCompositor.h" | 
| #include "core/rendering/RenderSelectionInfo.h" | 
| #include "core/rendering/RenderWidget.h" | 
| +#include "core/svg/SVGElement.h" | 
| namespace WebCore { | 
| @@ -273,6 +274,13 @@ void RenderView::layout() | 
| || child->style()->logicalMaxHeight().isViewportPercentage() | 
| || child->isSVGRoot()) | 
| layoutScope.setChildNeedsLayout(child); | 
| + | 
| + // SVG resource containers need to be invalidated outside any particular SVG root layout scope. | 
| 
pdr.
2013/09/23 22:26:02
This comment isn't actually helpful. Can you expan
 
f(malita)
2013/09/23 23:20:33
Didn't want to just duplicate the info in SVGRende
 | 
| + if (child->isSVGRoot()) { | 
| + ASSERT(child->node()); | 
| + ASSERT(child->node()->isSVGElement()); | 
| + toSVGElement(child->node())->invalidateRelativeLengthClients(&layoutScope); | 
| + } | 
| } | 
| } |