Support subpixel values for text-shadow and box-shadow

Source/core/rendering/style/ShadowData.h

Index: Source/core/rendering/style/ShadowData.h
diff --git a/Source/core/rendering/style/ShadowData.h b/Source/core/rendering/style/ShadowData.h
index 574b3f8214ec235b882831251c7e64439db4c2f7..6023a532a538420df4d0586b8d705be83a11dfe8 100644
--- a/Source/core/rendering/style/ShadowData.h
+++ b/Source/core/rendering/style/ShadowData.h
@@ -36,7 +36,7 @@ enum ShadowStyle { Normal, Inset };
 class ShadowData {
     WTF_MAKE_FAST_ALLOCATED;
 public:
-    ShadowData(const IntPoint& location, int blur, int spread, ShadowStyle style, const Color& color)
+    ShadowData(const LayoutPoint& location, LayoutUnit blur, LayoutUnit spread, ShadowStyle style, const Color& color)
         : m_location(location)
         , m_blur(blur)
         , m_spread(spread)
@@ -50,18 +50,18 @@ public:
 
     ShadowData blend(const ShadowData& from, double progress) const;
 
-    int x() const { return m_location.x(); }
-    int y() const { return m_location.y(); }
-    IntPoint location() const { return m_location; }
-    int blur() const { return m_blur; }
-    int spread() const { return m_spread; }
+    LayoutUnit x() const { return m_location.x(); }
+    LayoutUnit y() const { return m_location.y(); }
+    LayoutPoint location() const { return m_location; }
+    LayoutUnit blur() const { return m_blur; }
+    LayoutUnit spread() const { return m_spread; }
     ShadowStyle style() const { return m_style; }
     const Color& color() const { return m_color; }
 
 private:
-    IntPoint m_location;
-    int m_blur;
-    int m_spread;
+    LayoutPoint m_location;

[Timothy Loh, 2013/12/02 07:03:10]

I don't think we store LayoutUnits/etc. in style data? Is using
FloatPoint/float/float more appropriate here (this will also probably resolve
dstockwell's concern)?

[cbiesinger, 2013/12/18 02:09:12]

Ah, looks like you're right - we store Lengths or floats.

I turned these into floats, and that did indeed solve dstockwell's issue as
well! Thanks for pointing that out!

+    LayoutUnit m_blur;
+    LayoutUnit m_spread;
     Color m_color;
     ShadowStyle m_style;
 };