Fix an issue where hit detection could fail on rect and ellipse shapes

Source/core/rendering/svg/RenderSVGEllipse.h

 /*
  * Copyright (C) 2012 Google, Inc.
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 20 matching lines @@
 
 namespace blink {
 
 class RenderSVGEllipse final : public RenderSVGShape {
 public:
     explicit RenderSVGEllipse(SVGGraphicsElement*);
     virtual ~RenderSVGEllipse();
 
     virtual ShapeGeometryCodePath geometryCodePath() const override { return m_usePathFallback ? PathGeometry : EllipseGeometryFastPath; }
 
-private:
     virtual const char* renderName() const override { return "RenderSVGEllipse"; }
 
+protected:

[fs, 2014/12/04 13:10:35]

Please restore the access specifiers ('protected' makes little sense here
because this class is final.)

     virtual void updateShapeFromElement() override;
     virtual bool isShapeEmpty() const override { return m_usePathFallback ? RenderSVGShape::isShapeEmpty() : m_fillBoundingBox.isEmpty(); }
     virtual bool shapeDependentStrokeContains(const FloatPoint&) override;
     virtual bool shapeDependentFillContains(const FloatPoint&, const WindRule) const override;
+
+private:
     void calculateRadiiAndCenter();
+    bool hasContinuousStroke() const;
 
 private:
     FloatPoint m_center;
     FloatSize m_radii;
     bool m_usePathFallback;
 };
 
 }
 
 #endif