Title: [151286] trunk/Source/WebCore
Revision
151286
Author
[email protected]
Date
2013-06-06 13:40:50 -0700 (Thu, 06 Jun 2013)

Log Message

Fix formatting in BasicShapes.cpp
https://bugs.webkit.org/show_bug.cgi?id=117309

Patch by Bem Jones-Bey <[email protected]> on 2013-06-06
Reviewed by Alexandru Chiculita.

I reformatted the code in BasicShapes.cpp (bug 116734) to fix the
style issues, and I got some of the formatting wrong, which makes the
code much harder to read. This patch fixes that formatting.

No new tests, formatting clean up.

* rendering/style/BasicShapes.cpp:
(WebCore::BasicShapeRectangle::path): Fix formatting.
(WebCore::BasicShapeInsetRectangle::path): Fix formatting.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (151285 => 151286)


--- trunk/Source/WebCore/ChangeLog	2013-06-06 20:29:03 UTC (rev 151285)
+++ trunk/Source/WebCore/ChangeLog	2013-06-06 20:40:50 UTC (rev 151286)
@@ -1,3 +1,20 @@
+2013-06-06  Bem Jones-Bey  <[email protected]>
+
+        Fix formatting in BasicShapes.cpp
+        https://bugs.webkit.org/show_bug.cgi?id=117309
+
+        Reviewed by Alexandru Chiculita.
+
+        I reformatted the code in BasicShapes.cpp (bug 116734) to fix the
+        style issues, and I got some of the formatting wrong, which makes the
+        code much harder to read. This patch fixes that formatting.
+
+        No new tests, formatting clean up.
+
+        * rendering/style/BasicShapes.cpp:
+        (WebCore::BasicShapeRectangle::path): Fix formatting.
+        (WebCore::BasicShapeInsetRectangle::path): Fix formatting.
+
 2013-06-06  Christophe Dumez  <[email protected]>
 
         Clean up AddIncludesForType in JSC bindings generators

Modified: trunk/Source/WebCore/rendering/style/BasicShapes.cpp (151285 => 151286)


--- trunk/Source/WebCore/rendering/style/BasicShapes.cpp	2013-06-06 20:29:03 UTC (rev 151285)
+++ trunk/Source/WebCore/rendering/style/BasicShapes.cpp	2013-06-06 20:40:50 UTC (rev 151286)
@@ -54,12 +54,13 @@
 void BasicShapeRectangle::path(Path& path, const FloatRect& boundingBox)
 {
     ASSERT(path.isEmpty());
-    path.addRoundedRect(FloatRect(floatValueForLength(
-        m_x,
-        boundingBox.width()) + boundingBox.x(),
-        floatValueForLength(m_y, boundingBox.height()) + boundingBox.y(),
-        floatValueForLength(m_width, boundingBox.width()),
-        floatValueForLength(m_height, boundingBox.height())),
+    path.addRoundedRect(
+        FloatRect(
+            floatValueForLength(m_x, boundingBox.width()) + boundingBox.x(),
+            floatValueForLength(m_y, boundingBox.height()) + boundingBox.y(),
+            floatValueForLength(m_width, boundingBox.width()),
+            floatValueForLength(m_height, boundingBox.height())
+        ),
         FloatSize(
             m_cornerRadiusX.isUndefined() ? 0 : floatValueForLength(m_cornerRadiusX, boundingBox.width()),
             m_cornerRadiusY.isUndefined() ? 0 : floatValueForLength(m_cornerRadiusY, boundingBox.height())
@@ -185,11 +186,13 @@
     ASSERT(path.isEmpty());
     float left = floatValueForLength(m_left, boundingBox.width());
     float top = floatValueForLength(m_top, boundingBox.height());
-    path.addRoundedRect(FloatRect(
-        left + boundingBox.x(),
-        top + boundingBox.y(),
-        std::max<float>(boundingBox.width() - left - floatValueForLength(m_right, boundingBox.width()), 0),
-        std::max<float>(boundingBox.height() - top - floatValueForLength(m_bottom, boundingBox.height()), 0)),
+    path.addRoundedRect(
+        FloatRect(
+            left + boundingBox.x(),
+            top + boundingBox.y(),
+            std::max<float>(boundingBox.width() - left - floatValueForLength(m_right, boundingBox.width()), 0),
+            std::max<float>(boundingBox.height() - top - floatValueForLength(m_bottom, boundingBox.height()), 0)
+        ),
         FloatSize(
             m_cornerRadiusX.isUndefined() ? 0 : floatValueForLength(m_cornerRadiusX, boundingBox.width()),
             m_cornerRadiusY.isUndefined() ? 0 : floatValueForLength(m_cornerRadiusY, boundingBox.height())
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to