Changed type of border-width longhands from unsigned to float.

third_party/WebKit/Source/core/css/resolver/AnimatedStyleBuilder.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 78 matching lines @@
     if (value->isLength())
         return BorderImageLength(toAnimatableLength(value)->length(state.style()->effectiveZoom(), ValueRangeNonNegative));
     if (value->isDouble())
         return BorderImageLength(clampTo<double>(toAnimatableDouble(value)->toDouble(), 0));
     ASSERT(toAnimatableUnknown(value)->toCSSValueID() == CSSValueAuto);
     return Length(Auto);
 }
 
 template<typename T> T animatableValueClampTo(const AnimatableValue* value, T min = defaultMinimumForClamp<T>(), T max = defaultMaximumForClamp<T>())
 {
-    static_assert(std::is_integral<T>::value, "should use integral type T when rounding values");
     return clampTo<T>(roundForImpreciseConversion<T>(toAnimatableDouble(value)->toDouble()), min, max);
 }
 
-template<typename T> T animatableLineWidthClamp(const AnimatableValue* value)
+template<typename T> T animatableLineWidthClamp(const AnimatableValue* value, T min = defaultMinimumForClamp<T>(), T max = defaultMaximumForClamp<T>())

[alancutter (OOO until 2018), 2016/01/20 06:06:23]

No need for min/max parameters. Line widths are always non-negative so the min 0
can be hard coded and max is defaulted anyway.

 {
     double doubleValue = toAnimatableDouble(value)->toDouble();
     // This matches StyleBuilderConverter::convertLineWidth().
-    return (doubleValue > 0 && doubleValue < 1) ? 1 : animatableValueClampTo<T>(value);
+    return (doubleValue > 0 && doubleValue < 1) ? 1 : animatableValueClampTo<T>(value, min, max);
 }
 
 LengthBox animatableValueToLengthBox(const AnimatableValue* value, const StyleResolverState& state, ValueRange range = ValueRangeAll)
 {
     const AnimatableLengthBox* animatableLengthBox = toAnimatableLengthBox(value);
     return LengthBox(
         animatableValueToLength(animatableLengthBox->top(), state, range),
         animatableValueToLength(animatableLengthBox->right(), state, range),
         animatableValueToLength(animatableLengthBox->bottom(), state, range),
         animatableValueToLength(animatableLengthBox->left(), state, range));
@@ skipping 198 matching lines @@
         style->setBorderBottomColor(toAnimatableColor(value)->color());
         style->setVisitedLinkBorderBottomColor(toAnimatableColor(value)->visitedLinkColor());
         return;
     case CSSPropertyBorderBottomLeftRadius:
         style->setBorderBottomLeftRadius(animatableValueToLengthSize(value, state, ValueRangeNonNegative));
         return;
     case CSSPropertyBorderBottomRightRadius:
         style->setBorderBottomRightRadius(animatableValueToLengthSize(value, state, ValueRangeNonNegative));
         return;
     case CSSPropertyBorderBottomWidth:
-        style->setBorderBottomWidth(animatableLineWidthClamp<unsigned>(value));
+        style->setBorderBottomWidth(animatableLineWidthClamp<float>(value, 0));
         return;
     case CSSPropertyBorderImageOutset:
         style->setBorderImageOutset(animatableValueToBorderImageLengthBox(value, state));
         return;
     case CSSPropertyBorderImageSlice:
         style->setBorderImageSlices(animatableValueToLengthBox(toAnimatableLengthBoxAndBool(value)->box(), state, ValueRangeNonNegative));
         style->setBorderImageSlicesFill(toAnimatableLengthBoxAndBool(value)->flag());
         return;
     case CSSPropertyBorderImageSource:
         style->setBorderImageSource(state.styleImage(property, *toAnimatableImage(value)->toCSSValue()));
         return;
     case CSSPropertyBorderImageWidth:
         style->setBorderImageWidth(animatableValueToBorderImageLengthBox(value, state));
         return;
     case CSSPropertyBorderLeftColor:
         style->setBorderLeftColor(toAnimatableColor(value)->color());
         style->setVisitedLinkBorderLeftColor(toAnimatableColor(value)->visitedLinkColor());
         return;
     case CSSPropertyBorderLeftWidth:
-        style->setBorderLeftWidth(animatableLineWidthClamp<unsigned>(value));
+        style->setBorderLeftWidth(animatableLineWidthClamp<float>(value, 0));
         return;
     case CSSPropertyBorderRightColor:
         style->setBorderRightColor(toAnimatableColor(value)->color());
         style->setVisitedLinkBorderRightColor(toAnimatableColor(value)->visitedLinkColor());
         return;
     case CSSPropertyBorderRightWidth:
-        style->setBorderRightWidth(animatableLineWidthClamp<unsigned>(value));
+        style->setBorderRightWidth(animatableLineWidthClamp<float>(value, 0));
         return;
     case CSSPropertyBorderTopColor:
         style->setBorderTopColor(toAnimatableColor(value)->color());
         style->setVisitedLinkBorderTopColor(toAnimatableColor(value)->visitedLinkColor());
         return;
     case CSSPropertyBorderTopLeftRadius:
         style->setBorderTopLeftRadius(animatableValueToLengthSize(value, state, ValueRangeNonNegative));
         return;
     case CSSPropertyBorderTopRightRadius:
         style->setBorderTopRightRadius(animatableValueToLengthSize(value, state, ValueRangeNonNegative));
         return;
     case CSSPropertyBorderTopWidth:
-        style->setBorderTopWidth(animatableLineWidthClamp<unsigned>(value));
+        style->setBorderTopWidth(animatableLineWidthClamp<float>(value, 0));
         return;
     case CSSPropertyBottom:
         style->setBottom(animatableValueToLength(value, state));
         return;
     case CSSPropertyBoxShadow:
         style->setBoxShadow(toAnimatableShadow(value)->shadowList());
         return;
     case CSSPropertyClip:
         style->setClip(animatableValueToLengthBox(value, state));
         return;
@@ skipping 328 matching lines @@
     case CSSPropertyRy:
         style->setRy(animatableValueToLength(value, state, ValueRangeNonNegative));
         return;
 
     default:
         ASSERT_NOT_REACHED();
     }
 }
 
 } // namespace blink