Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1065)

Unified Diff: Source/core/rendering/svg/RenderSVGRect.h

Issue 745383007: Fix an issue where hit detection could fail on rect and ellipse shapes (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/rendering/svg/RenderSVGRect.h
diff --git a/Source/core/rendering/svg/RenderSVGRect.h b/Source/core/rendering/svg/RenderSVGRect.h
index 4703e4d23e30e587c68ab111d20074e4618365fc..9fbcf9e57ea615e2cd7890a31a942042eafd8707 100644
--- a/Source/core/rendering/svg/RenderSVGRect.h
+++ b/Source/core/rendering/svg/RenderSVGRect.h
@@ -39,15 +39,18 @@ public:
virtual ShapeGeometryCodePath geometryCodePath() const override { return m_usePathFallback ? PathGeometry : RectGeometryFastPath; }
-private:
virtual const char* renderName() const override { return "RenderSVGRect"; }
+protected:
fs 2014/12/04 13:10:35 Same as for RenderSVGEllipse.
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:
+ bool hasSimpleStroke() const;
+
+private:
FloatRect m_innerStrokeRect;
FloatRect m_outerStrokeRect;
bool m_usePathFallback;

Powered by Google App Engine
This is Rietveld 408576698