FloatingObject m_paginationStrut should be LayoutUnit
authorbjonesbe@adobe.com <bjonesbe@adobe.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 7 Feb 2014 18:37:55 +0000 (18:37 +0000)
committerbjonesbe@adobe.com <bjonesbe@adobe.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 7 Feb 2014 18:37:55 +0000 (18:37 +0000)
https://bugs.webkit.org/show_bug.cgi?id=119808

Reviewed by Andreas Kling.

Make the paginationStrut in FloatingObject have the same type as all
of the other paginationStruts in the code.

* rendering/FloatingObjects.h:
(WebCore::FloatingObject::paginationStrut):
(WebCore::FloatingObject::setPaginationStrut):
* rendering/RenderBlockLineLayout.cpp:
(WebCore::RenderBlockFlow::positionNewFloatOnLine):

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

Source/WebCore/ChangeLog
Source/WebCore/rendering/FloatingObjects.h
Source/WebCore/rendering/RenderBlockLineLayout.cpp

index 1694496..a0ebca8 100644 (file)
@@ -1,3 +1,19 @@
+2014-02-07  Bem Jones-Bey  <bjonesbe@adobe.com>
+
+        FloatingObject m_paginationStrut should be LayoutUnit
+        https://bugs.webkit.org/show_bug.cgi?id=119808
+
+        Reviewed by Andreas Kling.
+
+        Make the paginationStrut in FloatingObject have the same type as all
+        of the other paginationStruts in the code.
+
+        * rendering/FloatingObjects.h:
+        (WebCore::FloatingObject::paginationStrut):
+        (WebCore::FloatingObject::setPaginationStrut):
+        * rendering/RenderBlockLineLayout.cpp:
+        (WebCore::RenderBlockFlow::positionNewFloatOnLine):
+
 2014-02-06  Brady Eidson  <beidson@apple.com>
 
         IDB: storage/indexeddb/create-index-with-integer-keys.html fails
 2014-02-06  Brady Eidson  <beidson@apple.com>
 
         IDB: storage/indexeddb/create-index-with-integer-keys.html fails
index 8519237..7222e97 100644 (file)
@@ -67,8 +67,8 @@ public:
     const LayoutRect& frameRect() const { ASSERT(isPlaced()); return m_frameRect; }
     void setFrameRect(const LayoutRect& frameRect) { ASSERT(!isInPlacedTree()); m_frameRect = frameRect; }
 
     const LayoutRect& frameRect() const { ASSERT(isPlaced()); return m_frameRect; }
     void setFrameRect(const LayoutRect& frameRect) { ASSERT(!isInPlacedTree()); m_frameRect = frameRect; }
 
-    int paginationStrut() const { return m_paginationStrut; }
-    void setPaginationStrut(int strut) { m_paginationStrut = strut; }
+    LayoutUnit paginationStrut() const { return m_paginationStrut; }
+    void setPaginationStrut(LayoutUnit strut) { m_paginationStrut = strut; }
 
 #ifndef NDEBUG
     bool isInPlacedTree() const { return m_isInPlacedTree; }
 
 #ifndef NDEBUG
     bool isInPlacedTree() const { return m_isInPlacedTree; }
@@ -88,7 +88,7 @@ private:
     RenderBox& m_renderer;
     RootInlineBox* m_originatingLine;
     LayoutRect m_frameRect;
     RenderBox& m_renderer;
     RootInlineBox* m_originatingLine;
     LayoutRect m_frameRect;
-    int m_paginationStrut; // FIXME: This should be a LayoutUnit, since it's a vertical offset.
+    LayoutUnit m_paginationStrut;
 
     unsigned m_type : 2; // Type (left or right aligned)
     unsigned m_shouldPaint : 1;
 
     unsigned m_type : 2; // Type (left or right aligned)
     unsigned m_shouldPaint : 1;
index 43cf1cd..3537c6b 100644 (file)
@@ -2116,7 +2116,7 @@ bool RenderBlockFlow::positionNewFloatOnLine(FloatingObject* newFloat, FloatingO
     ASSERT(floatingObjectSet.last().get() == newFloat);
 
     LayoutUnit floatLogicalTop = logicalTopForFloat(newFloat);
     ASSERT(floatingObjectSet.last().get() == newFloat);
 
     LayoutUnit floatLogicalTop = logicalTopForFloat(newFloat);
-    int paginationStrut = newFloat->paginationStrut();
+    LayoutUnit paginationStrut = newFloat->paginationStrut();
 
     if (floatLogicalTop - paginationStrut != logicalHeight() + lineInfo.floatPaginationStrut())
         return true;
 
     if (floatLogicalTop - paginationStrut != logicalHeight() + lineInfo.floatPaginationStrut())
         return true;