[CSS Shapes] Simplify Polygon implementation

Source/platform/geometry/FloatRect.h

Index: Source/platform/geometry/FloatRect.h
diff --git a/Source/platform/geometry/FloatRect.h b/Source/platform/geometry/FloatRect.h
index 661ca88a9ecfef63abfe8f6dcc9a1c5c4101891e..7f68ab1a8897be05904419abbb4d83b949e000e2 100644
--- a/Source/platform/geometry/FloatRect.h
+++ b/Source/platform/geometry/FloatRect.h
@@ -142,6 +142,9 @@ public:
         return px >= x() && px <= maxX() && py >= y() && py <= maxY();
     }
 
+    bool overlapsYRange(float y1, float y2) const { return !isEmpty() && y2 >= y1 && y2 >= y() && y1 <= maxY(); }

[Stephen Chennney, 2014/04/04 20:42:35]

I don't think this should be in FloatRect.

It's checking facts about the range (y2 >= y1) in addition to the rect
comparison, and it's only used in the code you're changing. If you can find
other users of the same functionality then modify them too, otherwise put this
in the shapes code.

+    bool overlapsXRange(float x1, float x2) const { return !isEmpty() && x2 >= x1 && x2 >= x() && x1 <= maxX(); }

[Stephen Chennney, 2014/04/04 20:42:35]

Ditto.

+
     void inflateX(float dx)
     {
         m_location.setX(m_location.x() - dx);