Remove now-unneeded checks for eStyleUnit_Null. b=379741 r+sr=bzbarsky

layout/base/nsLayoutUtils.cpp

@@ -2185,8 +2185,6 @@ static PRBool NonZeroStyleCoord(const nsStyleCoord& aCoord)
     return aCoord.GetPercentValue() > 0;
   case eStyleUnit_Coord:
     return aCoord.GetCoordValue() > 0;
-  case eStyleUnit_Null:
-    return PR_FALSE;
   default:
     return PR_TRUE;
   }

layout/generic/nsAbsoluteContainingBlock.cpp

@@ -184,12 +184,10 @@ nsAbsoluteContainingBlock::Reflow(nsIFrame*                aDelegatingFrame,
 }
 
 static inline PRBool IsFixedPaddingSize(nsStyleUnit aUnit) {
-  return aUnit == eStyleUnit_Coord || aUnit == eStyleUnit_Null ||
-         aUnit == eStyleUnit_Chars;
+  return aUnit == eStyleUnit_Coord || aUnit == eStyleUnit_Chars;
 }
 static inline PRBool IsFixedMarginSize(nsStyleUnit aUnit) {
-  return aUnit == eStyleUnit_Coord || aUnit == eStyleUnit_Null ||
-         aUnit == eStyleUnit_Chars;
+  return aUnit == eStyleUnit_Coord || aUnit == eStyleUnit_Chars;
 }
 static inline PRBool IsFixedMaxSize(nsStyleUnit aUnit) {
   return aUnit == eStyleUnit_None || aUnit == eStyleUnit_Coord ||

layout/generic/nsBlockFrame.cpp

@@ -2463,16 +2463,14 @@ nsBlockFrame::AttributeChanged(PRInt32         aNameSpaceID,
 inline PRBool
 IsPaddingZero(nsStyleUnit aUnit, nsStyleCoord &aCoord)
 {
-    return (aUnit == eStyleUnit_Null ||
-            (aUnit == eStyleUnit_Coord && aCoord.GetCoordValue() == 0) ||
+    return ((aUnit == eStyleUnit_Coord && aCoord.GetCoordValue() == 0) ||
             (aUnit == eStyleUnit_Percent && aCoord.GetPercentValue() == 0.0));
 }
 
 inline PRBool
 IsMarginZero(nsStyleUnit aUnit, nsStyleCoord &aCoord)
 {
-    return (aUnit == eStyleUnit_Null ||
-            aUnit == eStyleUnit_Auto ||
+    return (aUnit == eStyleUnit_Auto ||
             (aUnit == eStyleUnit_Coord && aCoord.GetCoordValue() == 0) ||
             (aUnit == eStyleUnit_Percent && aCoord.GetPercentValue() == 0.0));
 }

layout/generic/nsInlineFrame.cpp

@@ -125,16 +125,14 @@ nsInlineFrame::GetType() const
 inline PRBool
 IsPaddingZero(nsStyleUnit aUnit, nsStyleCoord &aCoord)
 {
-    return (aUnit == eStyleUnit_Null ||
-            (aUnit == eStyleUnit_Coord && aCoord.GetCoordValue() == 0) ||
+    return ((aUnit == eStyleUnit_Coord && aCoord.GetCoordValue() == 0) ||
             (aUnit == eStyleUnit_Percent && aCoord.GetPercentValue() == 0.0));
 }
 
 inline PRBool
 IsMarginZero(nsStyleUnit aUnit, nsStyleCoord &aCoord)
 {
-    return (aUnit == eStyleUnit_Null ||
-            aUnit == eStyleUnit_Auto ||
+    return (aUnit == eStyleUnit_Auto ||
             (aUnit == eStyleUnit_Coord && aCoord.GetCoordValue() == 0) ||
             (aUnit == eStyleUnit_Percent && aCoord.GetPercentValue() == 0.0));
 }

layout/generic/nsLineLayout.cpp

@@ -2008,8 +2008,8 @@ nsLineLayout::VerticalAlignFrames(PerSpanData* psd)
       // Only consider non empty text frames when line-height=normal
       PRBool canUpdate = !pfd->GetFlag(PFD_ISTEXTFRAME);
       if (!canUpdate && pfd->GetFlag(PFD_ISNONWHITESPACETEXTFRAME)) {
-        nsStyleUnit lhUnit = frame->GetStyleText()->mLineHeight.GetUnit();
-        canUpdate = lhUnit == eStyleUnit_Normal || lhUnit == eStyleUnit_Null;
+        canUpdate =
+          frame->GetStyleText()->mLineHeight.GetUnit() == eStyleUnit_Normal;
       }
       if (canUpdate) {
 #endif

layout/style/nsStyleCoord.h

@@ -84,7 +84,10 @@ public:
   PRBool         operator==(const nsStyleCoord& aOther) const;
   PRBool         operator!=(const nsStyleCoord& aOther) const;
 
-  nsStyleUnit GetUnit(void) const { return mUnit; }
+  nsStyleUnit GetUnit(void) const {
+    NS_ASSERTION(mUnit != eStyleUnit_Null, "reading uninitialized value");
+    return mUnit;
+  }
   nscoord     GetCoordValue(void) const;
   PRInt32     GetIntValue(void) const;
   float       GetPercentValue(void) const;

layout/style/nsStyleStruct.cpp

@@ -63,8 +63,7 @@
 
 inline PRBool IsFixedUnit(nsStyleUnit aUnit, PRBool aEnumOK)
 {
-  return PRBool((aUnit == eStyleUnit_Null) || 
-                (aUnit == eStyleUnit_Coord) || 
+  return PRBool((aUnit == eStyleUnit_Coord) || 
                 (aEnumOK && (aUnit == eStyleUnit_Enumerated)));
 }
 
@@ -198,8 +197,6 @@ static nscoord CalcCoord(const nsStyleCoord& aCoord,
                          PRInt32 aNumEnums)
 {
   switch (aCoord.GetUnit()) {
-    case eStyleUnit_Null:
-      return 0;
     case eStyleUnit_Coord:
       return aCoord.GetCoordValue();
     case eStyleUnit_Enumerated: