Skip to content

GitLab

T
tangled
Commit fe3cf4eb authored by Vladimir Levin's avatar Vladimir Levin Committed by Commit Bot
Browse files
Options

content-visibility: Make the property not animatable. 

This patch makes the property not animatable. This goes with the
proposed issue here:
https://github.com/w3c/csswg-drafts/issues/5708

R=futhark@chromium.org, chrishtr@chromium.org

Fixed: 1146181
Change-Id: Ia352f2c5aab324b1426dd68c00d4d92ea0e760c6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526901Reviewed-by: default avatarChris Harrelson <chrishtr@chromium.org>
Reviewed-by: default avatarRune Lillesveen <futhark@chromium.org>
Commit-Queue: vmpstr <vmpstr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#825393}
parent d4119112
Hide whitespace changes
Inline Side-by-side
Showing with 28 additions and 0 deletions
third_party/blink/renderer/core/animation/css/css_animations.cc
View file @ fe3cf4eb
...@@ -1854,6 +1854,7 @@ bool CSSAnimations::IsAnimationAffectingProperty(const CSSProperty& property) { ...@@ -1854,6 +1854,7 @@ bool CSSAnimations::IsAnimationAffectingProperty(const CSSProperty& property) {
case CSSPropertyID::kAnimationPlayState: case CSSPropertyID::kAnimationPlayState:
case CSSPropertyID::kAnimationTimeline: case CSSPropertyID::kAnimationTimeline:
case CSSPropertyID::kAnimationTimingFunction: case CSSPropertyID::kAnimationTimingFunction:
case CSSPropertyID::kContentVisibility:
case CSSPropertyID::kContain: case CSSPropertyID::kContain:
case CSSPropertyID::kDirection: case CSSPropertyID::kDirection:
case CSSPropertyID::kDisplay: case CSSPropertyID::kDisplay:
......
third_party/blink/web_tests/external/wpt/css/css-contain/content-visibility/content-visibility-077.tentative.html 0 → 100644
View file @ fe3cf4eb
<!doctype HTML>
<meta charset="utf8">
<title>CSS Content Visibility: content-visibility is not animatable.</title>
<link rel="author" title="Vladimir Levin" href="mailto:vmpstr@chromium.org">
<link rel="help" href="https://drafts.csswg.org/css-contain/#content-visibility">
<meta name="assert" content="content-visibility is not animatable">
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
<style>
@keyframes cv {
from { content-visibility: auto }
to { content-visibility: hidden }
}
#container { animation: cv 1s; }
</style>
<div id=container></div>
<script>
test(() => {
const computedStyle = getComputedStyle(container);
assert_equals(computedStyle.contentVisibility, "visible");
}, "Content-visibility is not animatable");
</script>
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