CSS shapes support in Web Inspector

Source/core/rendering/shapes/PolygonShape.cpp

Index: Source/core/rendering/shapes/PolygonShape.cpp
diff --git a/Source/core/rendering/shapes/PolygonShape.cpp b/Source/core/rendering/shapes/PolygonShape.cpp
index 15fd9bd451b0b58abc4f7dfb467e7225cb9827b2..f9322acc7d529a8c8c1d2eafb29f2fc28a7e0eb6 100644
--- a/Source/core/rendering/shapes/PolygonShape.cpp
+++ b/Source/core/rendering/shapes/PolygonShape.cpp
@@ -148,4 +148,15 @@ void PolygonShape::getExcludedIntervals(LayoutUnit logicalTop, LayoutUnit logica
         result.append(LineSegment(excludedInterval.x1(), excludedInterval.x2()));
 }
 
+void PolygonShape::buildDisplayPaths(DisplayPaths& paths) const
+{
+    // FIXME: Add support for margin
+    if (!m_polygon.numberOfVertices())
+        return;
+    paths.shape.moveTo(m_polygon.vertexAt(0));
+    for (size_t i = 1; i < m_polygon.numberOfVertices(); i++)

[rwlbuis, 2014/04/15 15:48:21]

the prefix ++ is preferred. Depending on the compiler it is either quicker or
equally quick as the postfix, so there is not much risk in using the prefix one
always for simple for loops.

[Habib Virji, 2014/04/17 15:43:47]

Done.

+        paths.shape.addLineTo(m_polygon.vertexAt(i));
+    paths.shape.closeSubpath();
+}
+
 } // namespace WebCore