Title: [175177] trunk/Source/WebCore
Revision
175177
Author
[email protected]
Date
2014-10-24 13:59:58 -0700 (Fri, 24 Oct 2014)

Log Message

Replace INT_MIN/MAX / kFixedPointDenominator with intMin/MaxForLayoutUnit.
https://bugs.webkit.org/show_bug.cgi?id=138047

Reviewed by Andreas Kling.

No change in functionality.

* css/CSSPrimitiveValue.cpp:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (175176 => 175177)


--- trunk/Source/WebCore/ChangeLog	2014-10-24 20:26:44 UTC (rev 175176)
+++ trunk/Source/WebCore/ChangeLog	2014-10-24 20:59:58 UTC (rev 175177)
@@ -1,3 +1,14 @@
+2014-10-24  Zalan Bujtas  <[email protected]>
+
+        Replace INT_MIN/MAX / kFixedPointDenominator with intMin/MaxForLayoutUnit.
+        https://bugs.webkit.org/show_bug.cgi?id=138047
+
+        Reviewed by Andreas Kling.
+
+        No change in functionality.
+
+        * css/CSSPrimitiveValue.cpp:
+
 2014-10-24  Alexey Proskuryakov  <[email protected]>
 
         REGRESSION(r174877): WebProcess sends an empty Cookies HTTP header

Modified: trunk/Source/WebCore/css/CSSPrimitiveValue.cpp (175176 => 175177)


--- trunk/Source/WebCore/css/CSSPrimitiveValue.cpp	2014-10-24 20:26:44 UTC (rev 175176)
+++ trunk/Source/WebCore/css/CSSPrimitiveValue.cpp	2014-10-24 20:59:58 UTC (rev 175177)
@@ -61,8 +61,8 @@
 
 // Max/min values for CSS, needs to slightly smaller/larger than the true max/min values to allow for rounding without overflowing.
 // Subtract two (rather than one) to allow for values to be converted to float and back without exceeding the LayoutUnit::max.
-const int maxValueForCssLength = INT_MAX / kFixedPointDenominator - 2;
-const int minValueForCssLength = INT_MIN / kFixedPointDenominator + 2;
+const int maxValueForCssLength = intMaxForLayoutUnit - 2;
+const int minValueForCssLength = intMinForLayoutUnit + 2;
 
 static inline bool isValidCSSUnitTypeForDoubleConversion(CSSPrimitiveValue::UnitTypes unitType)
 {
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to