Title: [177628] trunk/Source/WebCore
Revision
177628
Author
[email protected]
Date
2014-12-21 17:52:23 -0800 (Sun, 21 Dec 2014)

Log Message

Fix missing assertion in r177623.
https://bugs.webkit.org/show_bug.cgi?id=139334

Reviewed by Darin Adler.

ValueWithCalculation::setCalculation() was meant to include an
"isCalculation()" assertion but I inadvertently omitted the
ASSERT() around the statement.

* css/CSSParser.h:
(WebCore::CSSParser::ValueWithCalculation::setCalculation):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (177627 => 177628)


--- trunk/Source/WebCore/ChangeLog	2014-12-21 23:56:55 UTC (rev 177627)
+++ trunk/Source/WebCore/ChangeLog	2014-12-22 01:52:23 UTC (rev 177628)
@@ -1,3 +1,17 @@
+2014-12-21  Chris Dumez  <[email protected]>
+
+        Fix missing assertion in r177623.
+        https://bugs.webkit.org/show_bug.cgi?id=139334
+
+        Reviewed by Darin Adler.
+
+        ValueWithCalculation::setCalculation() was meant to include an
+        "isCalculation()" assertion but I inadvertently omitted the
+        ASSERT() around the statement.
+
+        * css/CSSParser.h:
+        (WebCore::CSSParser::ValueWithCalculation::setCalculation):
+
 2014-12-21  Gabor Rapcsanyi  <[email protected]>
 
         AX: Hidden aria table crash

Modified: trunk/Source/WebCore/css/CSSParser.h (177627 => 177628)


--- trunk/Source/WebCore/css/CSSParser.h	2014-12-21 23:56:55 UTC (rev 177627)
+++ trunk/Source/WebCore/css/CSSParser.h	2014-12-22 01:52:23 UTC (rev 177628)
@@ -92,7 +92,7 @@
         CSSCalcValue* calculation() const { return m_calculation.get(); }
         void setCalculation(RefPtr<CSSCalcValue>&& calculation)
         {
-            isCalculation(m_value);
+            ASSERT(isCalculation(m_value));
             m_calculation = calculation;
         }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to