No max-width means "none", not null unit. Bug 381284, r+sr=dbaron
authorbzbarsky@mit.edu
Wed, 23 May 2007 12:56:02 -0700
changeset 1793 6f48c3aeba88f31332e5a0c7a7a09df370d62334
parent 1792 384c2f37c53a66dea7059cddedd7195b84714654
child 1794 ce6840837eaae8a3361c9f540d3d3dcfe41cf61d
push id1
push userroot
push dateTue, 26 Apr 2011 22:38:44 +0000
treeherdermozilla-beta@bfdb6e623a36 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs381284
milestone1.9a5pre
No max-width means "none", not null unit. Bug 381284, r+sr=dbaron
layout/tables/BasicTableLayoutStrategy.cpp
--- a/layout/tables/BasicTableLayoutStrategy.cpp
+++ b/layout/tables/BasicTableLayoutStrategy.cpp
@@ -154,17 +154,17 @@ GetWidthInfo(nsIRenderingContext *aRende
             default:
                 NS_NOTREACHED("unexpected enumerated value");
         }
     }
 
     nsStyleCoord maxWidth(aStylePos->mMaxWidth);
     if (maxWidth.GetUnit() == eStyleUnit_Enumerated) {
         if (!aIsCell || maxWidth.GetIntValue() == NS_STYLE_WIDTH_FILL)
-            maxWidth.Reset();
+            maxWidth.SetNoneValue();
         else if (maxWidth.GetIntValue() == NS_STYLE_WIDTH_SHRINK_WRAP)
             // for 'max-width', '-moz-shrink-wrap' is like
             // '-moz-intrinsic'
             maxWidth.SetIntValue(NS_STYLE_WIDTH_INTRINSIC,
                                  eStyleUnit_Enumerated);
     }
     unit = maxWidth.GetUnit();
     // XXX To really implement 'max-width' well, we'd need to store