Force all LayoutUnit construction to be explicit

third_party/WebKit/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 53 matching lines @@
 
 // TODO(thakis): Remove these two lines once http://llvm.org/PR26504 is resolved
 class LayoutUnit;
 inline bool operator<(const LayoutUnit&, const LayoutUnit&);
 
 class LayoutUnit {
     DISALLOW_NEW_EXCEPT_PLACEMENT_NEW();
 public:
     LayoutUnit() : m_value(0) { }
     // TODO(leviw): All of the below constructors should be explicit. crbug.com/581254
-    LayoutUnit(int value) { setValue(value); }
-    LayoutUnit(unsigned short value) { setValue(value); }
-    LayoutUnit(unsigned value) { setValue(value); }
-    LayoutUnit(unsigned long value) { m_value = clampTo<int>(value * kFixedPointDenominator); }
-    LayoutUnit(unsigned long long value) { m_value = clampTo<int>(value * kFixedPointDenominator); }
-    LayoutUnit(float value) { m_value = clampTo<int>(value * kFixedPointDenominator); }
-    LayoutUnit(double value) { m_value = clampTo<int>(value * kFixedPointDenominator); }
+    explicit LayoutUnit(int value) { setValue(value); }
+    explicit LayoutUnit(unsigned short value) { setValue(value); }
+    explicit LayoutUnit(unsigned value) { setValue(value); }
+    explicit LayoutUnit(unsigned long value) { m_value = clampTo<int>(value * kFixedPointDenominator); }
+    explicit LayoutUnit(unsigned long long value) { m_value = clampTo<int>(value * kFixedPointDenominator); }
+    explicit LayoutUnit(float value) { m_value = clampTo<int>(value * kFixedPointDenominator); }
+    explicit LayoutUnit(double value) { m_value = clampTo<int>(value * kFixedPointDenominator); }
 
     static LayoutUnit fromFloatCeil(float value)
     {
         LayoutUnit v;
         v.m_value = clampTo<int>(ceilf(value * kFixedPointDenominator));
         return v;
     }
 
     static LayoutUnit fromFloatFloor(float value)
     {
@@ skipping 698 matching lines @@
 inline LayoutUnit absoluteValue(const LayoutUnit& value)
 {
     return value.abs();
 }
 
 inline LayoutUnit layoutMod(const LayoutUnit& numerator, const LayoutUnit& denominator)
 {
     return numerator % denominator;
 }
 
+inline LayoutUnit layoutMod(const LayoutUnit& numerator, int denominator)
+{
+    return numerator % LayoutUnit(denominator);
+}
+
 inline bool isIntegerValue(const LayoutUnit value)
 {
     return value.toInt() == value;
 }
 
 inline LayoutUnit clampToLayoutUnit(LayoutUnit value, LayoutUnit min, LayoutUnit max)
 {
     if (value >= max)
         return max;
     if (value <= min)
         return min;
     return value;
 }
 
 } // namespace blink
 
 #endif // LayoutUnit_h