Fix pointer-events:all when stroke="none"

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 66 matching lines @@
     virtual ShapeGeometryCodePath geometryCodePath() const { return PathGeometry; }
     virtual const Vector<FloatPoint>* zeroLengthLineCaps() const { return 0; }
 
     virtual FloatRect objectBoundingBox() const override final { return m_fillBoundingBox; }
 
 protected:
     virtual void updateShapeFromElement();
     virtual bool shapeDependentStrokeContains(const FloatPoint&);
     virtual bool shapeDependentFillContains(const FloatPoint&, const WindRule) const;
 
+    virtual FloatRect actualStrokeBoundingBox() const { return style()->svgStyle().hasStroke() ? m_strokeBoundingBox : m_fillBoundingBox; }
+
     // Give RenderSVGPath a hook for updating markers in updateShapeFromElement.
     virtual void processMarkerPositions() { };
 
     FloatRect m_fillBoundingBox;
     FloatRect m_strokeBoundingBox;
 
 private:
     // Hit-detection separated for the fill and the stroke
     bool fillContains(const FloatPoint&, bool requiresFill = true, const WindRule fillRule = RULE_NONZERO);
     bool strokeContains(const FloatPoint&, bool requiresStroke = true);
 
     virtual const AffineTransform& localToParentTransform() const override final { return m_localTransform ? *m_localTransform : RenderSVGModelObject::localToParentTransform(); }
 
     virtual bool isOfType(RenderObjectType type) const override { return type == RenderObjectSVGShape || RenderSVGModelObject::isOfType(type); }
     virtual const char* renderName() const override { return "RenderSVGShape"; }
 
     virtual void layout() override final;
     virtual void paint(const PaintInfo&, const LayoutPoint&) override final;
     virtual void addFocusRingRects(Vector<LayoutRect>&, const LayoutPoint& additionalOffset) const override final;
 
     virtual bool nodeAtFloatPoint(const HitTestRequest&, HitTestResult&, const FloatPoint& pointInParent, HitTestAction) override final;
 
-    virtual FloatRect strokeBoundingBox() const override final { return m_strokeBoundingBox; }
+    virtual FloatRect strokeBoundingBox() const override final { return actualStrokeBoundingBox(); }

[fs, 2015/01/05 10:08:13]

This redefines the semantics of this method in this sub-tree of the class
hierarchy (and hence transitively affects containers.) Could we do the opposite
instead - ie. keep strokeBoundingBox as-is and add a new method to be used for
hit-testing?

     FloatRect calculateObjectBoundingBox() const;
     FloatRect calculateStrokeBoundingBox() const;
     void updatePaintInvalidationBoundingBox();
     void updateLocalTransform();
 
 private:
     OwnPtr<AffineTransform> m_localTransform;
     OwnPtr<Path> m_path;
 
     bool m_needsBoundariesUpdate : 1;
     bool m_needsShapeUpdate : 1;
     bool m_needsTransformUpdate : 1;
 };
 
 DEFINE_RENDER_OBJECT_TYPE_CASTS(RenderSVGShape, isSVGShape());
 
 }
 
 #endif