Changeset 112172 in webkit

Timestamp:

Mar 26, 2012 4:37:44 PM (12 years ago)

Author:

leviw@chromium.org

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):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/ColumnInfo.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-03-26  Levi Weintraub  <leviw@chromium.org>
+
+        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  <leviw@chromium.org>
 

trunk/Source/WebCore/rendering/ColumnInfo.h

@@ -84 +84 @@
         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;