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

Source/core/rendering/svg/RenderSVGShape.h

 /*
  * Copyright (C) 2004, 2005, 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005 Rob Buis <buis@kde.org>
  * Copyright (C) 2005 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2006 Apple Computer, Inc
  * Copyright (C) 2009 Google, Inc.
  * Copyright (C) 2011 Renata Hodovan <reni@webkit.org>
  * Copyright (C) 2011 University of Szeged
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 42 matching lines @@
     virtual void strokeShape(GraphicsContext*) const;
 
     bool nodeAtFloatPointInternal(const HitTestRequest&, const FloatPoint&, PointerEventsHitRules);
 
     Path& path() const
     {
         ASSERT(m_path);
         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(); }
     virtual bool shapeDependentStrokeContains(const FloatPoint&);
     virtual bool shapeDependentFillContains(const FloatPoint&, const WindRule) const;
     float strokeWidth() const;
     bool hasPath() const { return m_path.get(); }
     bool hasSmoothStroke() const;
 
     bool hasNonScalingStroke() const { return style()->svgStyle()->vectorEffect() == VE_NON_SCALING_STROKE; }
@@ skipping 46 matching lines @@
     bool m_needsBoundariesUpdate : 1;
     bool m_needsShapeUpdate : 1;
     bool m_needsTransformUpdate : 1;
 };
 
 DEFINE_RENDER_OBJECT_TYPE_CASTS(RenderSVGShape, isSVGShape());
 
 }
 
 #endif