Commit 446ac790 authored by Manish Goregaokar's avatar Manish Goregaokar
Browse files

Bug 1290864 - Omit the default closest-side when serializing circle() and ellipse(); r=xidorn

MozReview-Commit-ID: HKwOL0YAK5E
parent 72a5e88c
Loading
Loading
Loading
Loading
+15 −5
Original line number Diff line number Diff line
@@ -941,6 +941,15 @@ nsCSSValue::AppendCircleOrEllipseToString(nsCSSKeyword aFunctionId,

  bool hasRadii = array->Item(1).GetUnit() != eCSSUnit_Null;

  // closest-side is the default, so we don't need to
  // output it if all values are closest-side.
  if (array->Item(1).GetUnit() == eCSSUnit_Enumerated &&
      array->Item(1).GetIntValue() == NS_RADIUS_CLOSEST_SIDE &&
      (aFunctionId == eCSSKeyword_circle ||
       (array->Item(2).GetUnit() == eCSSUnit_Enumerated &&
        array->Item(2).GetIntValue() == NS_RADIUS_CLOSEST_SIDE))) {
    hasRadii = false;
  } else {
    AppendPositionCoordinateToString(array->Item(1), aProperty,
                                     aResult, aSerialization);

@@ -949,6 +958,7 @@ nsCSSValue::AppendCircleOrEllipseToString(nsCSSKeyword aFunctionId,
      AppendPositionCoordinateToString(array->Item(2), aProperty,
                                       aResult, aSerialization);
    }
  }

  // Any position specified?
  if (array->Item(count).GetUnit() != eCSSUnit_Array) {