Use larger values for no-minimum and no-maximum so that we don't cross them. (Bug 466395) r+sr=roc a191=beltzner
authorL. David Baron <dbaron@dbaron.org>
Thu, 27 Nov 2008 20:08:32 -0800
changeset 22068 11f15643c4036914dabd8c748b2b5b2bb2e90abc
parent 22067 78cc43260ab1329e0417082477b4e756fa65831c
child 22069 f77eb57bf63db0fdc570af078424f63ea760fd5d
push idunknown
push userunknown
push dateunknown
bugs466395
milestone1.9.1b3pre
Use larger values for no-minimum and no-maximum so that we don't cross them. (Bug 466395) r+sr=roc a191=beltzner
layout/generic/nsLineLayout.cpp
layout/reftests/bugs/466395-1-ref.html
layout/reftests/bugs/466395-1.html
layout/reftests/bugs/466395-2-ref.html
layout/reftests/bugs/466395-2.html
layout/reftests/bugs/reftest.list
--- a/layout/generic/nsLineLayout.cpp
+++ b/layout/generic/nsLineLayout.cpp
@@ -1518,18 +1518,18 @@ nsLineLayout::PlaceTopBottomFrames(PerSp
     }
     if (span) {
       nscoord distanceFromTop = aDistanceFromTop + pfd->mBounds.y;
       PlaceTopBottomFrames(span, distanceFromTop, aLineHeight);
     }
   }
 }
 
-#define VERTICAL_ALIGN_FRAMES_NO_MINIMUM 32767
-#define VERTICAL_ALIGN_FRAMES_NO_MAXIMUM -32768
+#define VERTICAL_ALIGN_FRAMES_NO_MINIMUM nscoord_MAX
+#define VERTICAL_ALIGN_FRAMES_NO_MAXIMUM nscoord_MIN
 
 // Vertically place frames within a given span. Note: this doesn't
 // place top/bottom aligned frames as those have to wait until the
 // entire line box height is known. This is called after the span
 // frame has finished being reflowed so that we know its height.
 void
 nsLineLayout::VerticalAlignFrames(PerSpanData* psd)
 {
new file mode 100644
--- /dev/null
+++ b/layout/reftests/bugs/466395-1-ref.html
@@ -0,0 +1,2 @@
+<title>Testcase, bug 466395</title>
+hello
new file mode 100644
--- /dev/null
+++ b/layout/reftests/bugs/466395-1.html
@@ -0,0 +1,8 @@
+<title>Testcase, bug 466395</title>
+<div style="overflow:hidden;height:500px;width:500px">
+  <div style="display: inline-block; height: 10px">
+    hello
+    <div style="height: 10000px"></div>
+    goodbye
+  </div>
+</div>
new file mode 100644
--- /dev/null
+++ b/layout/reftests/bugs/466395-2-ref.html
@@ -0,0 +1,3 @@
+<!DOCTYPE HTML>
+<title>Testcase, bug 466395</title>
+hello
new file mode 100644
--- /dev/null
+++ b/layout/reftests/bugs/466395-2.html
@@ -0,0 +1,9 @@
+<!DOCTYPE HTML>
+<title>Testcase, bug 466395</title>
+<div style="overflow:hidden;height:500px;width:500px">
+  <div style="display: inline-block; height: 10px">
+    hello
+    <div style="height: 10000px"></div>
+    goodbye
+  </div>
+</div>
--- a/layout/reftests/bugs/reftest.list
+++ b/layout/reftests/bugs/reftest.list
@@ -973,8 +973,10 @@ fails == 461512-1.html 461512-1-ref.html
 == 462844-1.html 462844-ref.html
 == 462844-2.html 462844-ref.html
 == 462844-3.html 462844-ref.html
 == 462844-4.html 462844-ref.html
 == 463204-1.html 463204-1-ref.html
 == 463217-1.xul 463217-1-ref.xul
 == 463952-1.html 463952-1-ref.html
 == 464811-1.html 464811-1-ref.html
+== 466395-1.html 466395-1-ref.html
+== 466395-2.html 466395-2-ref.html