Title: [124500] branches/chromium/1180/Source/WebCore/platform/FractionalLayoutUnit.h
Revision
124500
Author
[email protected]
Date
2012-08-02 14:34:51 -0700 (Thu, 02 Aug 2012)

Log Message

Merge 121709 - LayoutUnit::epsilon() is wrong
https://bugs.webkit.org/show_bug.cgi?id=90083

Patch by Behdad Esfahbod <[email protected]> on 2012-07-02
Reviewed by Eric Seidel.

Do division in floats, not integers.

No new tests. No code using the affected function.

* platform/FractionalLayoutUnit.h:
(WebCore::FractionalLayoutUnit::epsilon):

[email protected]
Review URL: https://chromiumcodereview.appspot.com/10828136

Modified Paths

Diff

Modified: branches/chromium/1180/Source/WebCore/platform/FractionalLayoutUnit.h (124499 => 124500)


--- branches/chromium/1180/Source/WebCore/platform/FractionalLayoutUnit.h	2012-08-02 21:28:12 UTC (rev 124499)
+++ branches/chromium/1180/Source/WebCore/platform/FractionalLayoutUnit.h	2012-08-02 21:34:51 UTC (rev 124500)
@@ -153,7 +153,7 @@
         return toInt();
     }
 
-    static float epsilon() { return 1 / kFixedPointDenominator; }
+    static float epsilon() { return 1.0f / kFixedPointDenominator; }
     static const FractionalLayoutUnit max()
     {
         FractionalLayoutUnit m;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to