[CSS Shapes] Simplify Polygon implementation

Source/platform/geometry/FloatRect.h

 /*
  * Copyright (C) 2003, 2006, 2007 Apple Inc.  All rights reserved.
  * Copyright (C) 2005 Nokia.  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 124 matching lines @@
     void uniteIfNonZero(const FloatRect&);
     void extend(const FloatPoint&);
 
     // Note, this doesn't match what IntRect::contains(IntPoint&) does; the int version
     // is really checking for containment of 1x1 rect, but that doesn't make sense with floats.
     bool contains(float px, float py) const
     {
         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);
         m_size.setWidth(m_size.width() + dx + dx);
     }
     void inflateY(float dy)
     {
         m_location.setY(m_location.y() - dy);
         m_size.setHeight(m_size.height() + dy + dy);
     }
@@ skipping 78 matching lines @@
 PLATFORM_EXPORT IntRect enclosedIntRect(const FloatRect&);
 
 PLATFORM_EXPORT IntRect roundedIntRect(const FloatRect&);
 
 // Map supplied rect from srcRect to an equivalent rect in destRect.
 PLATFORM_EXPORT FloatRect mapRect(const FloatRect&, const FloatRect& srcRect, const FloatRect& destRect);
 
 }
 
 #endif