Title: [113005] trunk/Source/WebCore
Revision
113005
Author
[email protected]
Date
2012-04-03 03:19:50 -0700 (Tue, 03 Apr 2012)

Log Message

2012-04-03  Nikolas Zimmermann  <[email protected]>

        Not reviewed. Attempt to fix chromium build.

        * svg/properties/SVGAnimatedEnumerationPropertyTearOff.h: Unsigned is always >= 0 - only check if its 0 here.
        (WebCore::SVGAnimatedEnumerationPropertyTearOff::setBaseVal):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (113004 => 113005)


--- trunk/Source/WebCore/ChangeLog	2012-04-03 09:57:19 UTC (rev 113004)
+++ trunk/Source/WebCore/ChangeLog	2012-04-03 10:19:50 UTC (rev 113005)
@@ -1,3 +1,10 @@
+2012-04-03  Nikolas Zimmermann  <[email protected]>
+
+        Not reviewed. Attempt to fix chromium build.
+
+        * svg/properties/SVGAnimatedEnumerationPropertyTearOff.h: Unsigned is always >= 0 - only check if its 0 here.
+        (WebCore::SVGAnimatedEnumerationPropertyTearOff::setBaseVal):
+
 2012-04-03  Alexander Pavlov  <[email protected]>
 
         Unreviewed, rolling out r112954 and r112969.

Modified: trunk/Source/WebCore/svg/properties/SVGAnimatedEnumerationPropertyTearOff.h (113004 => 113005)


--- trunk/Source/WebCore/svg/properties/SVGAnimatedEnumerationPropertyTearOff.h	2012-04-03 09:57:19 UTC (rev 113004)
+++ trunk/Source/WebCore/svg/properties/SVGAnimatedEnumerationPropertyTearOff.h	2012-04-03 10:19:50 UTC (rev 113005)
@@ -33,7 +33,7 @@
     virtual void setBaseVal(const unsigned& property, ExceptionCode& ec)
     {
         // All SVG enumeration values, that are allowed to be set via SVG DOM start with 1, 0 corresponds to unknown and is not settable through SVG DOM.
-        if (property <= 0 || property > SVGPropertyTraits<EnumType>::highestEnumValue()) {
+        if (!property || property > SVGPropertyTraits<EnumType>::highestEnumValue()) {
             ec = SVGException::SVG_INVALID_VALUE_ERR;
             return;
         }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to