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

Unified 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, 10 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/css/CSSProperties.json5
diff --git a/third_party/WebKit/Source/core/css/CSSProperties.json5 b/third_party/WebKit/Source/core/css/CSSProperties.json5
index ef2d5c6b6abc551c2746f037353d977a5b908222..3ce34b558f795f01e002136170bdf11f5da8f7f7 100644
--- a/third_party/WebKit/Source/core/css/CSSProperties.json5
+++ b/third_party/WebKit/Source/core/css/CSSProperties.json5
@@ -96,16 +96,16 @@
// - 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.
- // - repeated
- // The property supports a list of values.
+ // - separator
+ // The property supports a list of values, and when there is more than one,
+ // it is separated with this character.
// - supports_percentage
// The property supports percentage types.
typedom_types: {
default: [],
},
- repeated: {
- default: false,
- valid_type: "bool",
+ separator: {
+ valid_values: [",", " ", "/"],
},
supports_percentage: {
default: false,
@@ -260,7 +260,7 @@
custom_all: true,
keywords: ["normal", "reverse", "alternate", "alternate-reverse"],
priority: "Animation",
- repeated: true,
+ separator: ",",
},
{
name: "animation-duration",
@@ -277,7 +277,7 @@
custom_all: true,
keywords: ["infinite"],
priority: "Animation",
- repeated: true,
+ separator: ",",
},
{
name: "animation-name",
@@ -871,7 +871,7 @@
api_class: true,
api_methods: ["parseSingleValue"],
custom_all: true,
- repeated: true,
+ separator: ",",
typedom_types: ["Image"],
},
{

Powered by Google App Engine
This is Rietveld 408576698