Title: [112172] trunk/Source/WebCore
Revision
112172
Author
[email protected]
Date
2012-03-26 16:37:44 -0700 (Mon, 26 Mar 2012)

Log Message

Switch ColumnInfo::addForcedBreak to LayoutUnits
https://bugs.webkit.org/show_bug.cgi?id=82210

Reviewed by Eric Seidel.

Forced breaks are added by local offsets, which should be in LayoutUnits.

No new tests. No change in behavior.

* rendering/ColumnInfo.h:
(WebCore::ColumnInfo::addForcedBreak):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (112171 => 112172)


--- trunk/Source/WebCore/ChangeLog	2012-03-26 23:36:06 UTC (rev 112171)
+++ trunk/Source/WebCore/ChangeLog	2012-03-26 23:37:44 UTC (rev 112172)
@@ -1,5 +1,19 @@
 2012-03-26  Levi Weintraub  <[email protected]>
 
+        Switch ColumnInfo::addForcedBreak to LayoutUnits
+        https://bugs.webkit.org/show_bug.cgi?id=82210
+
+        Reviewed by Eric Seidel.
+
+        Forced breaks are added by local offsets, which should be in LayoutUnits.
+
+        No new tests. No change in behavior.
+
+        * rendering/ColumnInfo.h:
+        (WebCore::ColumnInfo::addForcedBreak):
+
+2012-03-26  Levi Weintraub  <[email protected]>
+
         Update LayoutUnit usage in ContainerNode, Element, and ElementRareData
         https://bugs.webkit.org/show_bug.cgi?id=82219
 

Modified: trunk/Source/WebCore/rendering/ColumnInfo.h (112171 => 112172)


--- trunk/Source/WebCore/rendering/ColumnInfo.h	2012-03-26 23:36:06 UTC (rev 112171)
+++ trunk/Source/WebCore/rendering/ColumnInfo.h	2012-03-26 23:37:44 UTC (rev 112172)
@@ -83,10 +83,10 @@
         m_maximumDistanceBetweenForcedBreaks = 0;
         m_forcedBreakOffset = 0;
     }
-    void addForcedBreak(int offsetFromFirstPage)
+    void addForcedBreak(LayoutUnit offsetFromFirstPage)
     { 
         ASSERT(!m_columnHeight);
-        int distanceFromLastBreak = offsetFromFirstPage - m_forcedBreakOffset;
+        LayoutUnit distanceFromLastBreak = offsetFromFirstPage - m_forcedBreakOffset;
         if (!distanceFromLastBreak)
             return;
         m_forcedBreaks++;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to