[SVG] Resources should be laid out in dependecy order.

Source/core/rendering/svg/SVGRenderSupport.cpp

 /*
  * Copyright (C) 2007, 2008 Rob Buis <buis@kde.org>
  * Copyright (C) 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2007 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2009 Google, Inc.  All rights reserved.
  * Copyright (C) 2009 Dirk Schulze <krit@webkit.org>
  * Copyright (C) Research In Motion Limited 2009-2010. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 224 matching lines @@
                         toRenderSVGText(child)->setNeedsTextMetricsUpdate();
                         toRenderSVGText(child)->setNeedsPositioningValuesUpdate();
                     }
 
                     needsLayout = true;
                 }
             }
         }
 
         SubtreeLayoutScope layoutScope(child);
-        if (needsLayout)
+        // Resource containers are nasty: they can invalidate clients outside the current SubtreeLayoutScope.
+        // Since they only care about viewport size changes (to resolve their relative lengths), we trigger
+        // their invalidation directly from SVGSVGElement::svgAttributeChange() or at a higher
+        // SubtreeLayoutScope (in RenderView::layout()).
+        if (needsLayout && !child->isSVGResourceContainer())
             layoutScope.setNeedsLayout(child);
 
+        layoutResourcesIfNeeded(child);
+
         if (child->needsLayout()) {
             child->layout();
             // Renderers are responsible for repainting themselves when changing, except
             // for the initial paint to avoid potential double-painting caused by non-sensical "old" bounds.
             // We could handle this in the individual objects, but for now it's easier to have
             // parent containers call repaint().  (RenderBlock::layout* has similar logic.)
             if (!childEverHadLayout)
                 child->repaint();
         } else if (layoutSizeChanged)
             notlayoutedObjects.add(child);
     }
 
     if (!layoutSizeChanged) {
         ASSERT(notlayoutedObjects.isEmpty());
         return;
     }
 
     // If the layout size changed, invalidate all resources of all children that didn't go through the layout() code path.
     HashSet<RenderObject*>::iterator end = notlayoutedObjects.end();
     for (HashSet<RenderObject*>::iterator it = notlayoutedObjects.begin(); it != end; ++it)
         invalidateResourcesOfChildren(*it);
 }
 
+void SVGRenderSupport::layoutResourcesIfNeeded(const RenderObject* object)
+{
+    ASSERT(object);
+
+    SVGResources* resources = SVGResourcesCache::cachedResourcesForRenderObject(object);
+    if (resources)
+        resources->layoutIfNeeded();
+}
+
 bool SVGRenderSupport::isOverflowHidden(const RenderObject* object)
 {
     // SVG doesn't support independent x/y overflow
     ASSERT(object->style()->overflowX() == object->style()->overflowY());
 
     // OSCROLL is never set for SVG - see StyleResolver::adjustRenderStyle
     ASSERT(object->style()->overflowX() != OSCROLL);
 
     // RenderSVGRoot should never query for overflow state - it should always clip itself to the initial viewport size.
     ASSERT(!object->isRoot());
@@ skipping 110 matching lines @@
 
 bool SVGRenderSupport::isEmptySVGInlineText(const RenderObject* object)
 {
     // RenderSVGInlineText performs whitespace filtering in order to support xml:space
     // (http://www.w3.org/TR/SVG/struct.html#LangSpaceAttrs), and can end up with an empty string
     // even when its original constructor argument is non-empty.
     return object->isSVGInlineText() && toRenderSVGInlineText(object)->hasEmptyText();
 }
 
 }