[SVG2] Elements that don't render shouldn't contribute to ancestor bboxes.

Source/core/rendering/svg/RenderSVGShape.h

Index: Source/core/rendering/svg/RenderSVGShape.h
diff --git a/Source/core/rendering/svg/RenderSVGShape.h b/Source/core/rendering/svg/RenderSVGShape.h
index 12a703f063b4f29d0e565710507ba66d9b7eb91d..7bea3320aed9a89b6ce9b0b1e5a39ab723813e14 100644
--- a/Source/core/rendering/svg/RenderSVGShape.h
+++ b/Source/core/rendering/svg/RenderSVGShape.h
@@ -60,6 +60,13 @@ public:
         return *m_path;
     }
 
+    virtual bool isRenderingDisabled() const

[fs, 2014/06/25 12:53:20]

The implementation of this is essentially the same as isShapeEmpty (which is
what's used as gating in paint) - so maybe they should be merged (I don't think
the fallback part of isShapeEmpty is needed anymore after the recent
bbox-fixes.)

[Erik Dahlström (inactive), 2014/06/25 14:57:42]

Done.

+    {
+        // Returning false by default ensures backwards compatibility for

[fs, 2014/06/25 12:53:20]

This will probably be mute w/ the above, but - backwards compatibility with
what? (i.e. what element doesn't implement...)

[Erik Dahlström (inactive), 2014/06/25 14:57:42]

Done.

+        // elements that don't implement this method.
+        return false;
+    }
+
 protected:
     virtual void updateShapeFromElement();
     virtual bool isShapeEmpty() const { return path().isEmpty(); }