Added initial_keyword flag to CSSProperties.in

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

 // This file specifies all the CSS properties we support and the necessary
 // information for our code generation. The various supported arguments
 // are described below with example usage
 
 
 // - alias_for=other-property
 // Properties specifying alias_for should be virtually identical to the
 // properties they alias. Minor parsing differences are allowed as long as
 // the CSSValues created are of the same format of the aliased property.
 
 
 // - runtime_flag=CSSGridLayout
 // The flag on RuntimeEnabledFeatures conditionally enables the property.
 // This doesn't currently work with alias_for.
 
 
 // - descriptor_only
 // These are actually descriptors and not CSS properties. Properties with
 // the same name as a descriptor do not get this flag.
 
 
 // - longhands=property;other-property
 // 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.
 
 
 // - keywords=[keyword1|keyword2]
 // This specifies all valid keyword values for the property.
 // TODO(sashab): Once all properties are represented here, delete
 // 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.
 // Keyword does not need to be specified as every property can take keywords.
 // - keywords=[keyword1|keyword2]
 // The property can take these keywords.
 // - supports_multiple
 // The property supports a list of values.
 // - supports_percentage
 // The property supports percentage types.
@@ skipping 144 matching lines @@
 border-top-right-radius interpolable, initial=initialBorderRadius, converter=convertRadius
 border-top-style type_name=EBorderStyle, initial=initialBorderStyle
 border-top-width interpolable, initial=initialBorderWidth, converter=convertLineWidth<unsigned>, typedom_types=[Length], keywords=[thin|medium|thick], supports_percentage
 bottom typedom_types=[Length], keywords=[auto], supports_percentage, interpolable, initial=initialOffset, converter=convertLengthOrAuto
 box-shadow interpolable, converter=convertShadow
 box-sizing
 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
 clip-rule inherited, svg, type_name=WindRule
 color-interpolation inherited, svg
 color-interpolation-filters inherited, svg, type_name=EColorInterpolation
 color-rendering inherited, svg
 column-fill type_name=ColumnFill
 contain runtime_flag=CSSContainment, converter=convertFlags<Containment>
 content custom_all, typedom_types=[Image], supports_multiple
 counter-increment custom_all
 counter-reset custom_all
 cursor inherited, custom_all
 cx interpolable, svg, converter=convertLength
 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
 filter interpolable, converter=convertFilterOperations
 flex-basis interpolable, converter=convertLengthOrAuto
 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
 grid-auto-flow runtime_flag=CSSGridLayout, converter=convertGridAutoFlow, type_name=GridAutoFlow
 grid-auto-rows runtime_flag=CSSGridLayout, converter=convertGridTrackSizeList
 grid-column-end runtime_flag=CSSGridLayout, converter=convertGridPosition
 grid-column-gap runtime_flag=CSSGridLayout, converter=convertLength
 grid-column-start runtime_flag=CSSGridLayout, converter=convertGridPosition
 grid-row-end runtime_flag=CSSGridLayout, converter=convertGridPosition
 grid-row-gap runtime_flag=CSSGridLayout, converter=convertLength
 grid-row-start runtime_flag=CSSGridLayout, converter=convertGridPosition
 grid-template-areas runtime_flag=CSSGridLayout, custom_all
 grid-template-columns runtime_flag=CSSGridLayout, custom_all
 grid-template-rows runtime_flag=CSSGridLayout, custom_all
 height typedom_types=[Length], keywords=[auto], supports_percentage, interpolable, initial=initialSize, converter=convertLengthSizing
 hyphens runtime_flag=CSSHyphens, inherited, type_name=Hyphens
 image-rendering inherited
 image-orientation runtime_flag=ImageOrientation, inherited, name_for_methods=RespectImageOrientation, converter=convertImageOrientation
 isolation
 justify-content initial=initialContentAlignment, converter=convertContentAlignmentData
 justify-items runtime_flag=CSSGridLayout, initial=initialSelfAlignment, converter=convertSelfOrDefaultAlignmentData
 justify-self runtime_flag=CSSGridLayout, initial=initialSelfAlignment, converter=convertSelfOrDefaultAlignmentData
 left typedom_types=[Length], keywords=[auto], supports_percentage, interpolable, initial=initialOffset, converter=convertLengthOrAuto
 letter-spacing interpolable, inherited, initial=initialLetterWordSpacing, converter=convertSpacing
 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
 margin-right interpolable, initial=initialMargin, converter=convertQuirkyLength
 margin-top interpolable, initial=initialMargin, converter=convertQuirkyLength
 marker-end inherited, svg, name_for_methods=MarkerEndResource, converter=convertFragmentIdentifier
 marker-mid inherited, svg, name_for_methods=MarkerMidResource, converter=convertFragmentIdentifier
 marker-start inherited, svg, name_for_methods=MarkerStartResource, converter=convertFragmentIdentifier
 mask svg, name_for_methods=MaskerResource, converter=convertFragmentIdentifier
 mask-source-type runtime_flag=CSSMaskSourceType, custom_all
@@ skipping 85 matching lines @@
 touch-action converter=convertFlags<TouchAction>, type_name=TouchAction
 transform typedom_types=[Transform], keywords=[none], interpolable, converter=convertTransformOperations
 transform-origin interpolable, converter=convertTransformOrigin
 transform-style name_for_methods=TransformStyle3D
 translate runtime_flag=CSSIndependentTransformProperties, converter=convertTranslate, interpolable
 rotate runtime_flag=CSSIndependentTransformProperties, converter=convertRotate, interpolable
 scale runtime_flag=CSSIndependentTransformProperties, converter=convertScale, interpolable
 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
 -webkit-app-region custom_all
 -webkit-background-clip use_handlers_for=CSSPropertyBackgroundClip
 -webkit-background-origin use_handlers_for=CSSPropertyBackgroundOrigin
 -webkit-border-horizontal-spacing interpolable, inherited, name_for_methods=HorizontalBorderSpacing, converter=convertComputedLength<short>
 -webkit-border-image initial=initialNinePieceImage, custom_value
 -webkit-border-vertical-spacing interpolable, inherited, name_for_methods=VerticalBorderSpacing, converter=convertComputedLength<short>
 -webkit-box-align type_name=EBoxAlignment
@@ skipping 232 matching lines @@
 -webkit-text-size-adjust alias_for=text-size-adjust
 -webkit-transform alias_for=transform
 -webkit-transform-origin alias_for=transform-origin
 -webkit-transform-style alias_for=transform-style
 -webkit-transition alias_for=transition
 -webkit-transition-delay alias_for=transition-delay
 -webkit-transition-duration alias_for=transition-duration
 -webkit-transition-property alias_for=transition-property
 -webkit-transition-timing-function alias_for=transition-timing-function
 -webkit-user-select alias_for=user-select