Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(165)

Side by Side Diff: third_party/WebKit/Source/core/css/CSSProperties.json5

Issue 2722503002: [CSS Typed OM] Specify and check separators for repeated properties (Closed)
Patch Set: Add "/" as a valid separator in CSSProperties.json5 Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 { 1 {
2 // This file specifies all the CSS properties we support and the necessary 2 // This file specifies all the CSS properties we support and the necessary
3 // information for our code generation. The various supported arguments 3 // information for our code generation. The various supported arguments
4 // are described below with example usage 4 // are described below with example usage
5 5
6 parameters: { 6 parameters: {
7 // - alias_for: "other-property" 7 // - alias_for: "other-property"
8 // Properties specifying alias_for should be virtually identical to the 8 // Properties specifying alias_for should be virtually identical to the
9 // properties they alias. Minor parsing differences are allowed as long as 9 // properties they alias. Minor parsing differences are allowed as long as
10 // the CSSValues created are of the same format of the aliased property. 10 // the CSSValues created are of the same format of the aliased property.
(...skipping 78 matching lines...) Expand 10 before | Expand all | Expand 10 after
89 89
90 // - initial_keyword: "keyword-value" 90 // - initial_keyword: "keyword-value"
91 // This specifies the initial keyword value for the keyword fields. 91 // This specifies the initial keyword value for the keyword fields.
92 initial_keyword: { 92 initial_keyword: {
93 }, 93 },
94 94
95 // Flags which go into CSSOMTypes: 95 // Flags which go into CSSOMTypes:
96 // - typedom_types: ["Type", "OtherType"] 96 // - typedom_types: ["Type", "OtherType"]
97 // The property can take types specified in typedom_types for CSS Typed OM. 97 // The property can take types specified in typedom_types for CSS Typed OM.
98 // Keyword does not need to be specified as every property can take keywords . 98 // Keyword does not need to be specified as every property can take keywords .
99 // - repeated 99 // - separator
100 // The property supports a list of values. 100 // The property supports a list of values, and when there is more than one,
101 // it is separated with this character.
101 // - supports_percentage 102 // - supports_percentage
102 // The property supports percentage types. 103 // The property supports percentage types.
103 typedom_types: { 104 typedom_types: {
104 default: [], 105 default: [],
105 }, 106 },
106 repeated: { 107 separator: {
107 default: false, 108 valid_values: [",", " ", "/"],
108 valid_type: "bool",
109 }, 109 },
110 supports_percentage: { 110 supports_percentage: {
111 default: false, 111 default: false,
112 valid_type: "bool", 112 valid_type: "bool",
113 }, 113 },
114 114
115 // Flags which go into CSSPropertyMetadata: 115 // Flags which go into CSSPropertyMetadata:
116 // - interpolable 116 // - interpolable
117 // The interpolable flag indicates whether a property can be animated smooth ly. 117 // The interpolable flag indicates whether a property can be animated smooth ly.
118 // If this flag is set, the property should also be added to the switch 118 // If this flag is set, the property should also be added to the switch
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
253 { 253 {
254 name: "animation-delay", 254 name: "animation-delay",
255 custom_all: true, 255 custom_all: true,
256 priority: "Animation", 256 priority: "Animation",
257 }, 257 },
258 { 258 {
259 name: "animation-direction", 259 name: "animation-direction",
260 custom_all: true, 260 custom_all: true,
261 keywords: ["normal", "reverse", "alternate", "alternate-reverse"], 261 keywords: ["normal", "reverse", "alternate", "alternate-reverse"],
262 priority: "Animation", 262 priority: "Animation",
263 repeated: true, 263 separator: ",",
264 }, 264 },
265 { 265 {
266 name: "animation-duration", 266 name: "animation-duration",
267 custom_all: true, 267 custom_all: true,
268 priority: "Animation", 268 priority: "Animation",
269 }, 269 },
270 { 270 {
271 name: "animation-fill-mode", 271 name: "animation-fill-mode",
272 custom_all: true, 272 custom_all: true,
273 priority: "Animation", 273 priority: "Animation",
274 }, 274 },
275 { 275 {
276 name: "animation-iteration-count", 276 name: "animation-iteration-count",
277 custom_all: true, 277 custom_all: true,
278 keywords: ["infinite"], 278 keywords: ["infinite"],
279 priority: "Animation", 279 priority: "Animation",
280 repeated: true, 280 separator: ",",
281 }, 281 },
282 { 282 {
283 name: "animation-name", 283 name: "animation-name",
284 custom_all: true, 284 custom_all: true,
285 priority: "Animation", 285 priority: "Animation",
286 }, 286 },
287 { 287 {
288 name: "animation-play-state", 288 name: "animation-play-state",
289 custom_all: true, 289 custom_all: true,
290 priority: "Animation", 290 priority: "Animation",
(...skipping 573 matching lines...) Expand 10 before | Expand all | Expand 10 after
864 api_class: true, 864 api_class: true,
865 api_methods: ["parseSingleValue"], 865 api_methods: ["parseSingleValue"],
866 converter: "convertFlags<Containment>", 866 converter: "convertFlags<Containment>",
867 runtime_flag: "CSSContainment", 867 runtime_flag: "CSSContainment",
868 }, 868 },
869 { 869 {
870 name: "content", 870 name: "content",
871 api_class: true, 871 api_class: true,
872 api_methods: ["parseSingleValue"], 872 api_methods: ["parseSingleValue"],
873 custom_all: true, 873 custom_all: true,
874 repeated: true, 874 separator: ",",
875 typedom_types: ["Image"], 875 typedom_types: ["Image"],
876 }, 876 },
877 { 877 {
878 name: "counter-increment", 878 name: "counter-increment",
879 api_class: true, 879 api_class: true,
880 custom_all: true, 880 custom_all: true,
881 }, 881 },
882 { 882 {
883 name: "counter-reset", 883 name: "counter-reset",
884 api_class: true, 884 api_class: true,
(...skipping 2188 matching lines...) Expand 10 before | Expand all | Expand 10 after
3073 { 3073 {
3074 name: "-webkit-transition-timing-function", 3074 name: "-webkit-transition-timing-function",
3075 alias_for: "transition-timing-function", 3075 alias_for: "transition-timing-function",
3076 }, 3076 },
3077 { 3077 {
3078 name: "-webkit-user-select", 3078 name: "-webkit-user-select",
3079 alias_for: "user-select", 3079 alias_for: "user-select",
3080 }, 3080 },
3081 ], 3081 ],
3082 } 3082 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698