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

Source/core/layout/svg/LayoutSVGShape.cpp

 /*
  * Copyright (C) 2004, 2005, 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2008 Rob Buis <buis@kde.org>
  * Copyright (C) 2005, 2007 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2009 Google, Inc.
  * Copyright (C) 2009 Dirk Schulze <krit@webkit.org>
  * Copyright (C) Research In Motion Limited 2010. All rights reserved.
  * Copyright (C) 2009 Jeff Schiller <codedread@gmail.com>
  * Copyright (C) 2011 Renata Hodovan <reni@webkit.org>
  * Copyright (C) 2011 University of Szeged
@@ skipping 55 matching lines @@
 }
 
 void LayoutSVGShape::updateShapeFromElement()
 {
     createPath();
 
     m_fillBoundingBox = calculateObjectBoundingBox();
     m_strokeBoundingBox = calculateStrokeBoundingBox();
 }
 
+FloatRect LayoutSVGShape::hitTestStrokeBoundingBox() const
+{
+    if (style()->svgStyle().hasStroke())
+        return m_strokeBoundingBox;
+
+    // Implementation of http://dev.w3.org/fxtf/css-masking-1/#compute-stroke-bounding-box
+    // for the <rect> / <ellipse> / <circle> case except that we ignore whether
+    // the stroke is none.
+
+    FloatRect box = m_fillBoundingBox;
+    const float strokeWidth = this->strokeWidth();
+    box.inflate(strokeWidth / 2);
+    return box;
+}
+
 bool LayoutSVGShape::shapeDependentStrokeContains(const FloatPoint& point)
 {
     ASSERT(m_path);
     StrokeData strokeData;
     SVGLayoutSupport::applyStrokeStyleToStrokeData(strokeData, styleRef(), *this);
 
     if (hasNonScalingStroke()) {
         AffineTransform nonScalingTransform = nonScalingStrokeTransform();
         Path* usePath = nonScalingStrokePath(m_path.get(), nonScalingTransform);
 
@@ skipping 14 matching lines @@
         return false;
 
     if (requiresFill && !SVGPaintServer::existsForLayoutObject(*this, styleRef(), ApplyToFillMode))
         return false;
 
     return shapeDependentFillContains(point, fillRule);
 }
 
 bool LayoutSVGShape::strokeContains(const FloatPoint& point, bool requiresStroke)
 {
-    if (!strokeBoundingBox().contains(point))
-        return false;
+    if (requiresStroke) {
+        if (!strokeBoundingBox().contains(point))
+            return false;
 
-    if (requiresStroke && !SVGPaintServer::existsForLayoutObject(*this, styleRef(), ApplyToStrokeMode))
-        return false;
+        if (!SVGPaintServer::existsForLayoutObject(*this, styleRef(), ApplyToStrokeMode))
+            return false;
+    } else {
+        if (!hitTestStrokeBoundingBox().contains(point))
+            return false;
+    }
 
     return shapeDependentStrokeContains(point);
 }
 
 void LayoutSVGShape::updateLocalTransform()
 {
     SVGGraphicsElement* graphicsElement = toSVGGraphicsElement(element());
     if (graphicsElement->hasAnimatedLocalTransform()) {
         if (m_localTransform)
             m_localTransform->setTransform(graphicsElement->calculateAnimatedLocalTransform());
@@ skipping 135 matching lines @@
     SVGLayoutSupport::intersectPaintInvalidationRectWithResources(this, m_paintInvalidationBoundingBox);
 }
 
 float LayoutSVGShape::strokeWidth() const
 {
     SVGLengthContext lengthContext(element());
     return lengthContext.valueForLength(style()->svgStyle().strokeWidth());
 }
 
 }