Unreviewed. Adding FIXME comment to FractionalLayoutUnit explaining the lack of size_...
authoreae@chromium.org <eae@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Apr 2012 18:46:57 +0000 (18:46 +0000)
committereae@chromium.org <eae@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Apr 2012 18:46:57 +0000 (18:46 +0000)
* platform/FractionalLayoutUnit.h:
(FractionalLayoutUnit):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114541 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/FractionalLayoutUnit.h

index fab697b..387cb20 100644 (file)
@@ -1,3 +1,10 @@
+2012-04-18  Emil A Eklund  <eae@chromium.org>
+
+        Unreviewed. Adding FIXME comment to FractionalLayoutUnit explaining the lack of size_t operators.
+
+        * platform/FractionalLayoutUnit.h:
+        (FractionalLayoutUnit):
+
 2012-04-18  Benjamin Poulain  <bpoulain@apple.com>
 
         Remove m_subclassData from JSArray, move the attribute to subclass as needed
index 927dacc..7f84cab 100644 (file)
@@ -44,6 +44,10 @@ const int intMinForLayoutUnit = -intMaxForLayoutUnit;
 
 class FractionalLayoutUnit {
 public:
+    // FIXME: Ideally we would have size_t versions of the constructor and operators.
+    // However due to compiler and platform differences adding those are non-trivial.
+    // See https://bugs.webkit.org/show_bug.cgi?id=83848 for details.
+    
     FractionalLayoutUnit() : m_value(0) { }
     FractionalLayoutUnit(int value) { ASSERT(isInBounds(value)); m_value = value * kFixedPointDenominator; }
     FractionalLayoutUnit(unsigned short value) { ASSERT(isInBounds(value)); m_value = value * kFixedPointDenominator; }