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

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 3fa24bf9487c95bb67f97e13d2f327c2b2e2106e..50fe6e53540f7f5ae46984a6975d589958175e9f 100644
--- a/Source/core/rendering/svg/RenderSVGShape.h
+++ b/Source/core/rendering/svg/RenderSVGShape.h
@@ -84,6 +84,8 @@ protected:
     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() { };
 
@@ -106,7 +108,7 @@ private:
 
     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();