Glyph positions maintain 32 bit integer part.

include/core/SkScalar.h

Index: include/core/SkScalar.h
diff --git a/include/core/SkScalar.h b/include/core/SkScalar.h
index 3c1787ed2da5ead04788b16ba50cbb8e9116b127..1b8c0dd4f01095b2865f5e991bff35026f96515a 100644
--- a/include/core/SkScalar.h
+++ b/include/core/SkScalar.h
@@ -169,11 +169,15 @@ static inline int SkDScalarRoundToInt(SkScalar x) {
 }
 
 static inline SkScalar SkScalarClampMax(SkScalar x, SkScalar max) {
-    return x < 0 ? 0 : x > max ? max : x;
+    x = SkTMin(x, max);
+    x = SkTMax(x, SkIntToScalar(0));

[reed1, 2015/03/09 16:33:56]

I actually think SkIntToScalar(0) is harder to read. I prefer just 0.

[bungeman-skia, 2015/03/09 17:49:12]

I went and pulled this change out into it's own review, since this CL doesn't
need it and it's an independent change. The issue here is that SkTMin and SkTMax
take (T, T) so just '0' (as an int) doesn't compile.

+    return x;
 }
 
 static inline SkScalar SkScalarPin(SkScalar x, SkScalar min, SkScalar max) {
-    return x < min ? min : x > max ? max : x;
+    x = SkTMin(x, max);
+    x = SkTMax(x, min);
+    return x;
 }
 
 SkScalar SkScalarSinCos(SkScalar radians, SkScalar* cosValue);