Changeset 173177 in webkit

Timestamp:

Sep 2, 2014 9:47:11 AM (10 years ago)

Author:

commit-queue@webkit.org

Message:

Unreviewed, rolling out r173175.
​https://bugs.webkit.org/show_bug.cgi?id=136454

it broke debug builds (Requested by jessieberlin on #webkit).

Reverted changeset:

"Introduce CSS_BASIC_TYPE_CASTS, and use it"
​https://bugs.webkit.org/show_bug.cgi?id=136403
​http://trac.webkit.org/changeset/173175

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• css/BasicShapeFunctions.cpp (modified) (4 diffs)
• css/CSSBasicShapes.cpp (modified) (4 diffs)
• css/CSSBasicShapes.h (modified) (5 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2014-09-02  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r173175.
+        https://bugs.webkit.org/show_bug.cgi?id=136454
+
+        it broke debug builds (Requested by jessieberlin on #webkit).
+
+        Reverted changeset:
+
+        "Introduce CSS_BASIC_TYPE_CASTS, and use it"
+        https://bugs.webkit.org/show_bug.cgi?id=136403
+        http://trac.webkit.org/changeset/173175
+
 2014-09-02  Alex Christensen  <achristensen@webkit.org>
 

trunk/Source/WebCore/css/BasicShapeFunctions.cpp

@@ -206 +206 @@
     switch (basicShapeValue->type()) {
     case CSSBasicShape::CSSBasicShapeCircleType: {
-        const CSSBasicShapeCircle* circleValue = toCSSBasicShapeCircle(basicShapeValue);
+        const CSSBasicShapeCircle* circleValue = static_cast<const CSSBasicShapeCircle *>(basicShapeValue);
         RefPtr<BasicShapeCircle> circle = BasicShapeCircle::create();
 
@@ -217 +217 @@
     }
     case CSSBasicShape::CSSBasicShapeEllipseType: {
-        const CSSBasicShapeEllipse* ellipseValue = toCSSBasicShapeEllipse(basicShapeValue);
+        const CSSBasicShapeEllipse* ellipseValue = static_cast<const CSSBasicShapeEllipse *>(basicShapeValue);
         RefPtr<BasicShapeEllipse> ellipse = BasicShapeEllipse::create();
 
@@ -230 +230 @@
     }
     case CSSBasicShape::CSSBasicShapePolygonType: {
-        const CSSBasicShapePolygon* polygonValue = toCSSBasicShapePolygon(basicShapeValue);
+        const CSSBasicShapePolygon* polygonValue = static_cast<const CSSBasicShapePolygon *>(basicShapeValue);
         RefPtr<BasicShapePolygon> polygon = BasicShapePolygon::create();
 
@@ -242 +242 @@
     }
     case CSSBasicShape::CSSBasicShapeInsetType: {
-        const CSSBasicShapeInset* rectValue = toCSSBasicShapeInset(basicShapeValue);
+        const CSSBasicShapeInset* rectValue = static_cast<const CSSBasicShapeInset* >(basicShapeValue);
         RefPtr<BasicShapeInset> rect = BasicShapeInset::create();
 

trunk/Source/WebCore/css/CSSBasicShapes.cpp

@@ -129 +129 @@
         return false;
 
-    const CSSBasicShapeCircle& other = toCSSBasicShapeCircle(shape);
+    const CSSBasicShapeCircle& other = static_cast<const CSSBasicShapeCircle&>(shape);
     return compareCSSValuePtr(m_centerX, other.m_centerX)
         && compareCSSValuePtr(m_centerY, other.m_centerY)
@@ -202 +202 @@
         return false;
 
-    const CSSBasicShapeEllipse& other = toCSSBasicShapeEllipse(shape);
+    const CSSBasicShapeEllipse& other = static_cast<const CSSBasicShapeEllipse&>(shape);
     return compareCSSValuePtr(m_centerX, other.m_centerX)
         && compareCSSValuePtr(m_centerY, other.m_centerY)
@@ -273 +273 @@
         return false;
 
-    const CSSBasicShapePolygon& rhs = toCSSBasicShapePolygon(shape);
+    const CSSBasicShapePolygon& rhs = static_cast<const CSSBasicShapePolygon&>(shape);
     return compareCSSValuePtr(m_referenceBox, rhs.m_referenceBox)
         && compareCSSValueVector<CSSPrimitiveValue>(m_values, rhs.m_values);
@@ -408 +408 @@
         return false;
 
-    const CSSBasicShapeInset& other = toCSSBasicShapeInset(shape);
+    const CSSBasicShapeInset& other = static_cast<const CSSBasicShapeInset&>(shape);
     return compareCSSValuePtr(m_top, other.m_top)
         && compareCSSValuePtr(m_right, other.m_right)

trunk/Source/WebCore/css/CSSBasicShapes.h

@@ -63 +63 @@
 };
 
-#define CSS_BASIC_TYPE_CASTS(ToValueTypeName, predicate) \
-    TYPE_CASTS_BASE(ToValueTypeName, CSSBasicShape, basicShape, basicShape->type() == predicate, basicShape.type() == predicate) 
-
 class CSSBasicShapeInset : public CSSBasicShape {
 public:
@@ -131 +128 @@
 };
 
-CSS_BASIC_TYPE_CASTS(CSSBasicShapeInset, CSSBasicShapeInsetType)
-
 class CSSBasicShapeCircle : public CSSBasicShape {
 public:
@@ -156 +151 @@
     RefPtr<CSSPrimitiveValue> m_radius;
 };
-
-CSS_BASIC_TYPE_CASTS(CSSBasicShapeCircle, CSSBasicShapeCircleType)
 
 class CSSBasicShapeEllipse : public CSSBasicShape {
@@ -186 +179 @@
 };
 
-CSS_BASIC_TYPE_CASTS(CSSBasicShapeEllipse, CSSBasicShapeEllipseType)
-
 class CSSBasicShapePolygon : public CSSBasicShape {
 public:
@@ -219 +210 @@
 };
 
-CSS_BASIC_TYPE_CASTS(CSSBasicShapePolygon, CSSBasicShapePolygonType)
-
 } // namespace WebCore