Bug 1637532 - Allow animations and transitions on ::marker; r=boris

As per the following change to the spec:

6b3d7240b5

The additional failure annotation is added because we don't currently treat
text-combine-upright as non-animatable and this patch exposes the existing bug
(bug 1654195).

Differential Revision: https://phabricator.services.mozilla.com/D84308
This commit is contained in:
Brian Birtles 2020-07-22 00:44:18 +00:00
Родитель 9159650f0b
Коммит 029c6ab57b
4 изменённых файлов: 46 добавлений и 101 удалений

Просмотреть файл

@ -1338,6 +1338,7 @@ var gCSSProperties = {
domProp: "animation",
inherited: false,
type: CSS_TYPE_TRUE_SHORTHAND,
applies_to_marker: true,
subproperties: [
"animation-name",
"animation-duration",
@ -1432,6 +1433,7 @@ var gCSSProperties = {
domProp: "animationDelay",
inherited: false,
type: CSS_TYPE_LONGHAND,
applies_to_marker: true,
initial_values: ["0s", "0ms"],
other_values: [
"1s",
@ -1447,6 +1449,7 @@ var gCSSProperties = {
domProp: "animationDirection",
inherited: false,
type: CSS_TYPE_LONGHAND,
applies_to_marker: true,
initial_values: ["normal"],
other_values: [
"alternate",
@ -1471,6 +1474,7 @@ var gCSSProperties = {
inherited: false,
type: CSS_TYPE_LONGHAND,
initial_values: ["0s", "0ms"],
applies_to_marker: true,
other_values: ["1s", "250ms", "1s, 250ms, 2.3s", "calc(1s + 2ms)"],
invalid_values: ["0", "0px", "-1ms", "-2s"],
},
@ -1478,6 +1482,7 @@ var gCSSProperties = {
domProp: "animationFillMode",
inherited: false,
type: CSS_TYPE_LONGHAND,
applies_to_marker: true,
initial_values: ["none"],
other_values: [
"forwards",
@ -1494,6 +1499,7 @@ var gCSSProperties = {
domProp: "animationIterationCount",
inherited: false,
type: CSS_TYPE_LONGHAND,
applies_to_marker: true,
initial_values: ["1"],
other_values: [
"infinite",
@ -1514,6 +1520,7 @@ var gCSSProperties = {
domProp: "animationName",
inherited: false,
type: CSS_TYPE_LONGHAND,
applies_to_marker: true,
initial_values: ["none"],
other_values: [
"all",
@ -1544,6 +1551,7 @@ var gCSSProperties = {
domProp: "animationPlayState",
inherited: false,
type: CSS_TYPE_LONGHAND,
applies_to_marker: true,
initial_values: ["running"],
other_values: [
"paused",
@ -1559,6 +1567,7 @@ var gCSSProperties = {
domProp: "animationTimingFunction",
inherited: false,
type: CSS_TYPE_LONGHAND,
applies_to_marker: true,
initial_values: ["ease"],
other_values: [
"cubic-bezier(0.25, 0.1, 0.25, 1.0)",
@ -8174,6 +8183,7 @@ var gCSSProperties = {
domProp: "transition",
inherited: false,
type: CSS_TYPE_TRUE_SHORTHAND,
applies_to_marker: true,
subproperties: [
"transition-property",
"transition-duration",
@ -8259,6 +8269,7 @@ var gCSSProperties = {
domProp: "transitionDelay",
inherited: false,
type: CSS_TYPE_LONGHAND,
applies_to_marker: true,
initial_values: ["0s", "0ms"],
other_values: ["1s", "250ms", "-100ms", "-1s", "1s, 250ms, 2.3s"],
invalid_values: ["0", "0px"],
@ -8267,6 +8278,7 @@ var gCSSProperties = {
domProp: "transitionDuration",
inherited: false,
type: CSS_TYPE_LONGHAND,
applies_to_marker: true,
initial_values: ["0s", "0ms"],
other_values: ["1s", "250ms", "1s, 250ms, 2.3s"],
invalid_values: ["0", "0px", "-1ms", "-2s"],
@ -8275,6 +8287,7 @@ var gCSSProperties = {
domProp: "transitionProperty",
inherited: false,
type: CSS_TYPE_LONGHAND,
applies_to_marker: true,
initial_values: ["all"],
other_values: [
"none",
@ -8314,6 +8327,7 @@ var gCSSProperties = {
domProp: "transitionTimingFunction",
inherited: false,
type: CSS_TYPE_LONGHAND,
applies_to_marker: true,
initial_values: ["ease"],
other_values: [
"cubic-bezier(0.25, 0.1, 0.25, 1.0)",
@ -9943,6 +9957,7 @@ var gCSSProperties = {
domProp: "MozTransition",
inherited: false,
type: CSS_TYPE_TRUE_SHORTHAND,
applies_to_marker: true,
alias_for: "transition",
subproperties: [
"transition-property",
@ -9955,6 +9970,7 @@ var gCSSProperties = {
domProp: "MozTransitionDelay",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "transition-delay",
subproperties: ["transition-delay"],
},
@ -9962,6 +9978,7 @@ var gCSSProperties = {
domProp: "MozTransitionDuration",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "transition-duration",
subproperties: ["transition-duration"],
},
@ -9969,6 +9986,7 @@ var gCSSProperties = {
domProp: "MozTransitionProperty",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "transition-property",
subproperties: ["transition-property"],
},
@ -9976,6 +9994,7 @@ var gCSSProperties = {
domProp: "MozTransitionTimingFunction",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "transition-timing-function",
subproperties: ["transition-timing-function"],
},
@ -9983,6 +10002,7 @@ var gCSSProperties = {
domProp: "MozAnimation",
inherited: false,
type: CSS_TYPE_TRUE_SHORTHAND,
applies_to_marker: true,
alias_for: "animation",
subproperties: [
"animation-name",
@ -9999,6 +10019,7 @@ var gCSSProperties = {
domProp: "MozAnimationDelay",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-delay",
subproperties: ["animation-delay"],
},
@ -10006,6 +10027,7 @@ var gCSSProperties = {
domProp: "MozAnimationDirection",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-direction",
subproperties: ["animation-direction"],
},
@ -10013,6 +10035,7 @@ var gCSSProperties = {
domProp: "MozAnimationDuration",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-duration",
subproperties: ["animation-duration"],
},
@ -10020,6 +10043,7 @@ var gCSSProperties = {
domProp: "MozAnimationFillMode",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-fill-mode",
subproperties: ["animation-fill-mode"],
},
@ -10027,6 +10051,7 @@ var gCSSProperties = {
domProp: "MozAnimationIterationCount",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-iteration-count",
subproperties: ["animation-iteration-count"],
},
@ -10034,6 +10059,7 @@ var gCSSProperties = {
domProp: "MozAnimationName",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-name",
subproperties: ["animation-name"],
},
@ -10041,6 +10067,7 @@ var gCSSProperties = {
domProp: "MozAnimationPlayState",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-play-state",
subproperties: ["animation-play-state"],
},
@ -10048,6 +10075,7 @@ var gCSSProperties = {
domProp: "MozAnimationTimingFunction",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-timing-function",
subproperties: ["animation-timing-function"],
},
@ -10965,6 +10993,7 @@ var gCSSProperties = {
domProp: "webkitAnimation",
inherited: false,
type: CSS_TYPE_TRUE_SHORTHAND,
applies_to_marker: true,
alias_for: "animation",
subproperties: [
"animation-name",
@ -10981,6 +11010,7 @@ var gCSSProperties = {
domProp: "webkitAnimationDelay",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-delay",
subproperties: ["animation-delay"],
},
@ -10988,6 +11018,7 @@ var gCSSProperties = {
domProp: "webkitAnimationDirection",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-direction",
subproperties: ["animation-direction"],
},
@ -10995,6 +11026,7 @@ var gCSSProperties = {
domProp: "webkitAnimationDuration",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-duration",
subproperties: ["animation-duration"],
},
@ -11002,6 +11034,7 @@ var gCSSProperties = {
domProp: "webkitAnimationFillMode",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-fill-mode",
subproperties: ["animation-fill-mode"],
},
@ -11009,6 +11042,7 @@ var gCSSProperties = {
domProp: "webkitAnimationIterationCount",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-iteration-count",
subproperties: ["animation-iteration-count"],
},
@ -11016,6 +11050,7 @@ var gCSSProperties = {
domProp: "webkitAnimationName",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-name",
subproperties: ["animation-name"],
},
@ -11023,6 +11058,7 @@ var gCSSProperties = {
domProp: "webkitAnimationPlayState",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-play-state",
subproperties: ["animation-play-state"],
},
@ -11030,6 +11066,7 @@ var gCSSProperties = {
domProp: "webkitAnimationTimingFunction",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "animation-timing-function",
subproperties: ["animation-timing-function"],
},
@ -11186,6 +11223,7 @@ var gCSSProperties = {
domProp: "webkitTransition",
inherited: false,
type: CSS_TYPE_TRUE_SHORTHAND,
applies_to_marker: true,
alias_for: "transition",
subproperties: [
"transition-property",
@ -11198,6 +11236,7 @@ var gCSSProperties = {
domProp: "webkitTransitionDelay",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "transition-delay",
subproperties: ["transition-delay"],
},
@ -11205,6 +11244,7 @@ var gCSSProperties = {
domProp: "webkitTransitionDuration",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "transition-duration",
subproperties: ["transition-duration"],
},
@ -11212,6 +11252,7 @@ var gCSSProperties = {
domProp: "webkitTransitionProperty",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "transition-property",
subproperties: ["transition-property"],
},
@ -11219,6 +11260,7 @@ var gCSSProperties = {
domProp: "webkitTransitionTimingFunction",
inherited: false,
type: CSS_TYPE_SHORTHAND_AND_LONGHAND,
applies_to_marker: true,
alias_for: "transition-timing-function",
subproperties: ["transition-timing-function"],
},

Просмотреть файл

@ -778,7 +778,9 @@ class PropertyRestrictions:
"direction",
"content",
"-moz-osx-font-smoothing",
] + PropertyRestrictions.spec(data, "css-fonts"))
] + PropertyRestrictions.spec(data, "css-fonts")
+ PropertyRestrictions.spec(data, "css-animations")
+ PropertyRestrictions.spec(data, "css-transitions"))
# https://www.w3.org/TR/webvtt1/#the-cue-pseudo-element
@staticmethod

Просмотреть файл

@ -1,85 +1,22 @@
[marker-supported-properties-in-animation.html]
[Animation of content in ::marker]
expected: FAIL
[Animation of font-style in ::marker]
expected: FAIL
[Animation of font-variant-east-asian in ::marker]
expected: FAIL
[Transition of white-space in ::marker]
expected: FAIL
[Animation of font-stretch in ::marker]
expected: FAIL
[Animation of font-family in ::marker]
expected: FAIL
[Transition of font-size-adjust in ::marker]
expected: FAIL
[Animation of font-kerning in ::marker]
expected: FAIL
[Animation of font-variant-caps in ::marker]
expected: FAIL
[Animation of font-size-adjust in ::marker]
expected: FAIL
[Transition of font-stretch in ::marker]
expected: FAIL
[Animation of color in ::marker]
expected: FAIL
[Transition of font in ::marker]
expected: FAIL
[Animation of font-variant-numeric in ::marker]
expected: FAIL
[Animation of list-style in ::marker]
expected: FAIL
[Animation of font in ::marker]
expected: FAIL
[Animation of font-variant in ::marker]
expected: FAIL
[Transition of list-style in ::marker]
expected: FAIL
[Animation of font-feature-settings in ::marker]
expected: FAIL
[Animation of font-weight in ::marker]
expected: FAIL
[Animation of font-synthesis in ::marker]
expected: FAIL
[Animation of white-space in ::marker]
expected: FAIL
[Animation of font-size in ::marker]
expected: FAIL
[Transition of font-weight in ::marker]
expected: FAIL
[Animation of font-variant-position in ::marker]
expected: FAIL
[Animation of font-variant-ligatures in ::marker]
expected: FAIL
[Transition of font-size in ::marker]
expected: FAIL
[Transition of color in ::marker]
[Animation of text-combine-upright in ::marker]
expected: FAIL

Просмотреть файл

@ -11,42 +11,6 @@
[Property animation value '1s linear 2s infinite alternate forwards paused anim' in ::marker]
expected: FAIL
[Property transition-duration value '2s' in ::marker]
expected: FAIL
[Property transition value 'display 1s linear 2s' in ::marker]
expected: FAIL
[Property animation-delay value '1s' in ::marker]
expected: FAIL
[Property transition-timing-function value 'linear' in ::marker]
expected: FAIL
[Property animation-fill-mode value 'forwards' in ::marker]
expected: FAIL
[Property animation-play-state value 'paused' in ::marker]
expected: FAIL
[Property animation-direction value 'alternate' in ::marker]
expected: FAIL
[Property animation-timing-function value 'linear' in ::marker]
expected: FAIL
[Property animation-duration value '2s' in ::marker]
expected: FAIL
[Property transition-delay value '1s' in ::marker]
expected: FAIL
[Property animation-name value 'anim' in ::marker]
expected: FAIL
[Property transition-property value 'display' in ::marker]
expected: FAIL
[Property animation-iteration-count value 'infinite' in ::marker]
expected: FAIL