Bug 1569701 Part 0 - Use nscoord instead of int for contentBEnd in nsColumnSetFrame::ReflowChildren(). r=dbaron
authorTing-Yu Lin <tlin@mozilla.com>
Wed, 14 Aug 2019 23:17:06 +0000
changeset 488286 e56e6ed91e5af309cbca9ce38252f03530d81b79
parent 488285 1b097ae05490a3b2c5e1c125160b0adea3515fe2
child 488287 6d5655c8638d8683fe22521583bddd575c668575
push id113906
push userncsoregi@mozilla.com
push dateFri, 16 Aug 2019 04:07:24 +0000
treeherdermozilla-inbound@d887276421d3 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdbaron
bugs1569701
milestone70.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1569701 Part 0 - Use nscoord instead of int for contentBEnd in nsColumnSetFrame::ReflowChildren(). r=dbaron Differential Revision: https://phabricator.services.mozilla.com/D41905
layout/generic/nsColumnSetFrame.cpp
--- a/layout/generic/nsColumnSetFrame.cpp
+++ b/layout/generic/nsColumnSetFrame.cpp
@@ -619,17 +619,17 @@ nsColumnSetFrame::ColumnBalanceData nsCo
       childOrigin.I(wm) =
           containerSize.width - borderPadding.Left(wm) - availISize;
 
       COLUMN_SET_LOG("%s: childOrigin.iCoord=%d", __func__, childOrigin.I(wm));
     }
   }
 
   int columnCount = 0;
-  int contentBEnd = 0;
+  nscoord contentBEnd = 0;
   bool reflowNext = false;
 
   while (child) {
     // Try to skip reflowing the child. We can't skip if the child is dirty. We
     // also can't skip if the next column is dirty, because the next column's
     // first line(s) might be pullable back to this column. We can't skip if
     // it's the last child because we need to obtain the bottom margin. We can't
     // skip if this is the last column and we're supposed to assign unbounded