Use clampTo instead of chaining std::max(std::min(...))

third_party/WebKit/Source/platform/graphics/filters/FEComponentTransfer.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005 Rob Buis <buis@kde.org>
  * Copyright (C) 2005 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2009 Dirk Schulze <krit@webkit.org>
  * Copyright (C) Research In Motion Limited 2010. All rights reserved.
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 10 matching lines @@
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  */
 
 #include "platform/graphics/filters/FEComponentTransfer.h"
 
 #include "SkColorFilterImageFilter.h"
 #include "SkTableColorFilter.h"
 #include "platform/graphics/filters/SkiaImageFilterBuilder.h"
 #include "platform/text/TextStream.h"
+#include "wtf/MathExtras.h"
+#include <algorithm>
 
 namespace blink {
 
 typedef void (*TransferType)(unsigned char*, const ComponentTransferFunction&);
 
 FEComponentTransfer::FEComponentTransfer(Filter* filter, const ComponentTransferFunction& redFunc, const ComponentTransferFunction& greenFunc,
     const ComponentTransferFunction& blueFunc, const ComponentTransferFunction& alphaFunc)
     : FilterEffect(filter)
     , m_redFunc(redFunc)
     , m_greenFunc(greenFunc)
@@ skipping 17 matching lines @@
     const Vector<float>& tableValues = transferFunction.tableValues;
     unsigned n = tableValues.size();
     if (n < 1)
         return;
     for (unsigned i = 0; i < 256; ++i) {
         double c = i / 255.0;
         unsigned k = static_cast<unsigned>(c * (n - 1));
         double v1 = tableValues[k];
         double v2 = tableValues[std::min((k + 1), (n - 1))];
         double val = 255.0 * (v1 + (c * (n - 1) - k) * (v2 - v1));
-        val = std::max(0.0, std::min(255.0, val));
+        val = clampTo(val, 0.0, 255.0);
         values[i] = static_cast<unsigned char>(val);
     }
 }
 
 static void discrete(unsigned char* values, const ComponentTransferFunction& transferFunction)
 {
     const Vector<float>& tableValues = transferFunction.tableValues;
     unsigned n = tableValues.size();
     if (n < 1)
         return;
     for (unsigned i = 0; i < 256; ++i) {
         unsigned k = static_cast<unsigned>((i * n) / 255.0);
         k = std::min(k, n - 1);
         double val = 255 * tableValues[k];
-        val = std::max(0.0, std::min(255.0, val));
+        val = clampTo(val, 0.0, 255.0);
         values[i] = static_cast<unsigned char>(val);
     }
 }
 
 static void linear(unsigned char* values, const ComponentTransferFunction& transferFunction)
 {
     for (unsigned i = 0; i < 256; ++i) {
         double val = transferFunction.slope * i + 255 * transferFunction.intercept;
-        val = std::max(0.0, std::min(255.0, val));
+        val = clampTo(val, 0.0, 255.0);
         values[i] = static_cast<unsigned char>(val);
     }
 }
 
 static void gamma(unsigned char* values, const ComponentTransferFunction& transferFunction)
 {
     for (unsigned i = 0; i < 256; ++i) {
         double exponent = transferFunction.exponent; // RCVT doesn't like passing a double and a float to pow, so promote this to double
         double val = 255.0 * (transferFunction.amplitude * pow((i / 255.0), exponent) + transferFunction.offset);
-        val = std::max(0.0, std::min(255.0, val));
+        val = clampTo(val, 0.0, 255.0);
         values[i] = static_cast<unsigned char>(val);
     }
 }
 
 bool FEComponentTransfer::affectsTransparentPixels()
 {
     double intercept = 0;
     switch (m_alphaFunc.type) {
     case FECOMPONENTTRANSFER_TYPE_UNKNOWN:
     case FECOMPONENTTRANSFER_TYPE_IDENTITY:
@@ skipping 88 matching lines @@
     ts << "{green: " << m_greenFunc << "}\n";
     writeIndent(ts, indent + 2);
     ts << "{blue: " << m_blueFunc << "}\n";
     writeIndent(ts, indent + 2);
     ts << "{alpha: " << m_alphaFunc << "}]\n";
     inputEffect(0)->externalRepresentation(ts, indent + 1);
     return ts;
 }
 
 } // namespace blink