Refactor AnimatedStyleBuilder.cpp

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 85 matching lines @@
     return BorderImageLength(toAnimatableLength(value)->getLength(
         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>()) {
+T roundedClampTo(double value) {
   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);
+  return clampTo<T>(roundForImpreciseConversion<T>(value));
 }
 
 template <typename T>
+T animatableValueClampTo(const AnimatableValue* value) {
+  return roundedClampTo<T>(toAnimatableDouble(value)->toDouble());
+}
+
+template <typename T>
 T animatableLineWidthClamp(const AnimatableValue* value) {
   double doubleValue = toAnimatableDouble(value)->toDouble();
   // This matches StyleBuilderConverter::convertLineWidth().
   return (doubleValue > 0 && doubleValue < 1)
              ? 1
              : animatableValueClampTo<T>(value);
 }
 
 LengthBox animatableValueToLengthBox(const AnimatableValue* value,
                                      const StyleResolverState& state,
@@ skipping 705 matching lines @@
       style->setRy(
           animatableValueToLength(value, state, ValueRangeNonNegative));
       return;
 
     default:
       ASSERT_NOT_REACHED();
   }
 }
 
 }  // namespace blink