Saturated arithmetic changed to use templates to fix build-issues

Source/platform/LayoutUnit.h

 /*
  * Copyright (c) 2012, 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 204 matching lines @@
     {
         return value <= static_cast<unsigned>(std::numeric_limits<int>::max()) / kFixedPointDenominator;
     }
     static bool isInBounds(double value)
     {
         return ::fabs(value) <= std::numeric_limits<int>::max() / kFixedPointDenominator;
     }
 
     ALWAYS_INLINE void setValue(int value)
     {
-        m_value = saturatedSet(value, kLayoutUnitFractionalBits);
+        // The inner function needs to be implemented as a template to
+        // maintain a common interface for both C++ and ARM versions. The first
+        // parameter specifies how many bits we saturate to (for ARM).
+        m_value = saturatedSetSigned<
+            32-kLayoutUnitFractionalBits,

[Sami, 2014/07/09 13:59:26]

Please add spaces around the '-'.

[picksi, 2014/07/09 14:27:52]

Done.

+            kLayoutUnitFractionalBits>(value);
     }
 
-    inline void setValue(unsigned value)
+    ALWAYS_INLINE void setValue(unsigned value)
     {
-        m_value = saturatedSet(value, kLayoutUnitFractionalBits);
+        // The inner function needs to be implemented as a template to
+        // maintain a common interface for both C++ and ARM versions. The first
+        // parameter specifies how many bits we saturate to (for ARM). This is
+        // one bit less than the signed version due to the ARM instructions for
+        // unsigned saturation.
+        m_value = saturatedSetUnsigned<
+            31-kLayoutUnitFractionalBits,

[Sami, 2014/07/09 13:59:26]

Ditto.

[picksi, 2014/07/09 14:27:52]

Done.

+            kLayoutUnitFractionalBits>(value);
     }
 
     int m_value;
 };
 
 inline bool operator<=(const LayoutUnit& a, const LayoutUnit& b)
 {
     return a.rawValue() <= b.rawValue();
 }
 
@@ skipping 547 matching lines @@
     if (value >= max)
         return max;
     if (value <= min)
         return min;
     return value;
 }
 
 } // namespace WebCore
 
 #endif // LayoutUnit_h