Layout support for new circle shape syntax

Source/core/css/BasicShapeFunctions.cpp

 /*
  * Copyright (C) 2012 Adobe Systems Incorporated. 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.
@@ skipping 338 matching lines @@
     }
     default:
         break;
     }
 
     if (basicShapeValue->layoutBox())
         basicShape->setLayoutBox(LayoutBox(*basicShapeValue->layoutBox()));
 
     return basicShape.release();
 }
+
+FloatPoint floatPointForCenterCoordinate(const BasicShapeCenterCoordinate& centerX, const BasicShapeCenterCoordinate& centerY, FloatSize boxSize)
+{
+    FloatPoint p;
+    float offset = floatValueForLength(centerX.length(), boxSize.width());
+    switch (centerX.keyword()) {

[leviw_travelin_and_unemployed, 2014/01/08 23:07:43]

I guess what I was imagining was this single accessor that took a FloatSize like
you have here since the function is always called with a pair of coordinates,
and it could call out to a static function that does the logic you're
duplicating here.

Anyways this is okay.

+    case BasicShapeCenterCoordinate::None:
+    case BasicShapeCenterCoordinate::Left:
+        p.setX(offset);
+        break;
+    case BasicShapeCenterCoordinate::Right:
+        p.setX(boxSize.width() - offset);
+        break;
+    default:
+        ASSERT_NOT_REACHED();
+    }
+
+    offset = floatValueForLength(centerY.length(), boxSize.height());
+    switch (centerY.keyword()) {
+    case BasicShapeCenterCoordinate::None:
+    case BasicShapeCenterCoordinate::Top:
+        p.setY(offset);
+        break;
+    case BasicShapeCenterCoordinate::Bottom:
+        p.setY(boxSize.height() - offset);
+        break;
+    default:
+        ASSERT_NOT_REACHED();
+    }
+
+    return p;
 }
+
+}