Underline Thickness is not uniform

Source/core/rendering/style/RenderStyleConstants.h

Index: Source/core/rendering/style/RenderStyleConstants.h
diff --git a/Source/core/rendering/style/RenderStyleConstants.h b/Source/core/rendering/style/RenderStyleConstants.h
index 010e0845980ec7edf00c000cadd9e7b23236063f..66cef18ca0a3375ea44228e5b7ca5ccb140d803b 100644
--- a/Source/core/rendering/style/RenderStyleConstants.h
+++ b/Source/core/rendering/style/RenderStyleConstants.h
@@ -363,6 +363,7 @@ inline TextDecoration operator| (TextDecoration a, TextDecoration b) { return Te
 inline TextDecoration& operator|= (TextDecoration& a, TextDecoration b) { return a = a | b; }
 
 enum TextDecorationStyle {
+    TextDecorationStyleNone,

[andersr, 2014/03/19 13:18:12]

Are you sure you need this?

The decoration style can't be "none" per spec, and I don't think the value is
needed internally either. Existence of decorations is controlled by
text-decoration-line (enum TextDecoration), not by text-decoration-style.

See http://www.w3.org/TR/css-text-decor-3/#text-decoration-style-property

[h.joshi, 2014/03/19 13:59:54]

Yes as per CSS doc "none" is not a defined value, I used it  as initial value
when we do not know the decoration style. Assigning one of the values defined
does not looked good to me.
Pls suggest

On 2014/03/19 13:18:12, rude wrote:

[andersr, 2014/03/19 14:39:44]

The correct value in that case is TextDecorationStyleSolid. (See spec).

(Although I suggest that you drop this chunk, and don't mess with
TextDecorationStyle at all in this patch. See other comment).

     TextDecorationStyleSolid,
     TextDecorationStyleDouble,
     TextDecorationStyleDotted,