Commit e57f5612 authored by George Steel's avatar George Steel Committed by Commit Bot

Fix test of non-animatable properties.

Follow-up to https://chromium-review.googlesource.com/c/chromium/src/+/2303353
addressing post-submission comments.

Add missed test case and fix list sorting.

Bug: 1105144
Change-Id: I307065b3a8f4915a8b400c328ef2998fa8305e7c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2314296Reviewed-by: default avatarOriol Brufau <obrufau@igalia.com>
Reviewed-by: default avatarKevin Ellis <kevers@chromium.org>
Commit-Queue: George Steel <gtsteel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#791933}
parent 74fcfa08
...@@ -1785,6 +1785,7 @@ bool CSSAnimations::IsAnimationAffectingProperty(const CSSProperty& property) { ...@@ -1785,6 +1785,7 @@ bool CSSAnimations::IsAnimationAffectingProperty(const CSSProperty& property) {
case CSSPropertyID::kContain: case CSSPropertyID::kContain:
case CSSPropertyID::kDirection: case CSSPropertyID::kDirection:
case CSSPropertyID::kDisplay: case CSSPropertyID::kDisplay:
case CSSPropertyID::kTextCombineUpright:
case CSSPropertyID::kTextOrientation: case CSSPropertyID::kTextOrientation:
case CSSPropertyID::kTransition: case CSSPropertyID::kTransition:
case CSSPropertyID::kTransitionDelay: case CSSPropertyID::kTransitionDelay:
...@@ -1795,7 +1796,6 @@ bool CSSAnimations::IsAnimationAffectingProperty(const CSSProperty& property) { ...@@ -1795,7 +1796,6 @@ bool CSSAnimations::IsAnimationAffectingProperty(const CSSProperty& property) {
case CSSPropertyID::kWebkitWritingMode: case CSSPropertyID::kWebkitWritingMode:
case CSSPropertyID::kWillChange: case CSSPropertyID::kWillChange:
case CSSPropertyID::kWritingMode: case CSSPropertyID::kWritingMode:
case CSSPropertyID::kTextCombineUpright:
return true; return true;
default: default:
return false; return false;
......
...@@ -38,6 +38,7 @@ const gNonAnimatableProps = [ ...@@ -38,6 +38,7 @@ const gNonAnimatableProps = [
'contain', 'contain',
'direction', 'direction',
'display', 'display',
'textCombineUpright',
'textOrientation', 'textOrientation',
'unicodeBidi', 'unicodeBidi',
'willChange', 'willChange',
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment