Added initial_keyword flag to CSSProperties.in

third_party/WebKit/Source/core/css/CSSProperties.in

Index: third_party/WebKit/Source/core/css/CSSProperties.in
diff --git a/third_party/WebKit/Source/core/css/CSSProperties.in b/third_party/WebKit/Source/core/css/CSSProperties.in
index fbe02bbf4e3d97f6f7adb39eb1073064f1696bdc..3053e16694675604f596eee021bbd61fe62a70c1 100644
--- a/third_party/WebKit/Source/core/css/CSSProperties.in
+++ b/third_party/WebKit/Source/core/css/CSSProperties.in
@@ -23,11 +23,11 @@
 // The property is a shorthand for several other properties.
 
 
-// keyword_only
+// - keyword_only
 // These properties only store keyword values. This is used when
 // generating the ComputedStyle storage for the property. The initial
-// value for this property on a ComputedStyle is the first keyword in
-// the specified list of keywords.
+// value for this property on a ComputedStyle is specified with the
+// initial_keyword flag below.
 // TODO(sashab, meade): Remove this once TypedOM types are specified for
 // every property, since this value can be inferred from that.
 
@@ -38,6 +38,11 @@
 // CSSValueKeywords.in and use this list instead.
 
 
+// - initial_keyword
+// This specifies the initial keyword value for the keyword_only
+// property.
+
+
 // Flags which go into CSSOMTypes:
 // - typedom_types=[Type|OtherType]
 // The property can take types specified in typedom_types for CSS Typed OM.
@@ -202,7 +207,7 @@ break-after type_name=EBreak
 break-before type_name=EBreak
 break-inside type_name=EBreak
 buffered-rendering svg
-caption-side inherited, keyword_only, keywords=[top|bottom|left|right]
+caption-side inherited, keyword_only, keywords=[top|bottom|left|right], initial_keyword=top
 clear
 clip interpolable, converter=convertClip, custom_all
 clip-path interpolable, converter=convertClipPath
@@ -221,7 +226,7 @@ cy interpolable, svg, converter=convertLength
 d interpolable, svg, converter=convertPathOrNone
 display
 dominant-baseline inherited, svg
-empty-cells inherited, type_name=EEmptyCells, keyword_only, keywords=[show|hide]
+empty-cells inherited, type_name=EEmptyCells, keyword_only, keywords=[show|hide], initial_keyword=show
 fill interpolable, inherited, svg, setter=setFillPaint, custom_all
 fill-opacity interpolable, inherited, svg, converter=convertNumberOrPercentage
 fill-rule inherited, svg, type_name=WindRule
@@ -231,7 +236,7 @@ flex-direction
 flex-grow interpolable, type_name=float
 flex-shrink interpolable, type_name=float
 flex-wrap
-float type_name=EFloat, name_for_methods=Floating, keyword_only, keywords=[none|left|right]
+float type_name=EFloat, name_for_methods=Floating, keyword_only, keywords=[none|left|right], initial_keyword=none
 flood-color interpolable, svg, converter=convertColor
 flood-opacity interpolable, svg, converter=convertNumberOrPercentage
 grid-auto-columns runtime_flag=CSSGridLayout, converter=convertGridTrackSizeList
@@ -259,7 +264,7 @@ letter-spacing interpolable, inherited, initial=initialLetterWordSpacing, conver
 lighting-color interpolable, svg, converter=convertColor
 line-height interpolable, inherited, getter=specifiedLineHeight, converter=convertLineHeight
 list-style-image interpolable, inherited, custom_value, typedom_types=[Image]
-list-style-position inherited, keyword_only, keywords=[outside|inside]
+list-style-position inherited, keyword_only, keywords=[outside|inside], initial_keyword=outside
 list-style-type inherited
 margin-bottom interpolable, initial=initialMargin, converter=convertQuirkyLength
 margin-left interpolable, initial=initialMargin, converter=convertQuirkyLength
@@ -365,7 +370,7 @@ scale runtime_flag=CSSIndependentTransformProperties, converter=convertScale, in
 unicode-bidi
 vector-effect svg
 vertical-align interpolable, custom_inherit, custom_value
-visibility interpolable, inherited, independent, keyword_only, keywords=[visible|hidden|collapse]
+visibility interpolable, inherited, independent, keyword_only, keywords=[visible|hidden|collapse], initial_keyword=visible

[napper, 2016/12/01 02:10:02]

In every case the initial keyword is simply the first in the list. Is there any
reason why you don't want to make that the initial keyword is specified? Is
there any case then the initial keyword is not the first in the list?

[sashab, 2016/12/01 02:35:21]

Yes - for text-transform (which just landed, so this patch has been rebased onto
it) I moved "none" back to being the last keyword since thats the order it was
in before, but none is the default value. Also, in general, this set should not
be order-sensitive - the ordering reflects some meaningful order as
self-documentation, not the strict order in which the keywords are applied or
something.

Also I feel even if it *is* always the first keyword in the list, it's much more
self-documenting and easier for other scripts to use if its explicitly
specified. Imagine having to change the default by taking a random value from
the middle of the list and moving it to the front... Not good. :)

ALSO (hmm im pretty good at these) later we will want to specify initial values
for non-keywords, so maybe we could rename this to initial_value, and if it's a
keyword_only property its a keyword, otherwise initial=Vector<Foo> would be like
Vector<Foo>()... Not sure... Possibilities are endless 8-)

 x interpolable, svg, converter=convertLength
 y interpolable, svg, converter=convertLength
 -webkit-appearance type_name=ControlPart