| OLD | NEW |
|---|
| (Empty) |
| 1 <script src="../../resources/testharness.js"></script> | |
| 2 <script src="../../resources/testharnessreport.js"></script> | |
| 3 <body></body> | |
| 4 <script> | |
| 5 function createTarget() { | |
| 6 return document.body.appendChild(document.createElement('div')); | |
| 7 } | |
| 8 | |
| 9 function setTaintedValue(target, property, value) { | |
| 10 target.animate({[property]: value}, {fill: 'forwards'}); | |
| 11 assert_equals(getComputedStyle(target).getPropertyValue(property), value, | |
| 12 `Tainted value ${value} set on ${property} by animation`); | |
| 13 } | |
| 14 | |
| 15 function testTaintedSubstitution(target, varValue, {animationName, customPropert
y}) { | |
| 16 target.style.animationName = varValue; | |
| 17 target.style.setProperty('--taint-accepted', varValue); | |
| 18 assert_equals(getComputedStyle(target).animationName, animationName, | |
| 19 `${varValue} with tainted values ommitted`); | |
| 20 assert_equals(getComputedStyle(target).getPropertyValue('--taint-accepted'), c
ustomProperty, | |
| 21 `${varValue} with tainted values accepted`); | |
| 22 } | |
| 23 | |
| 24 test(() => { | |
| 25 var target = createTarget(); | |
| 26 setTaintedValue(target, '--tainted', 'tainted'); | |
| 27 testTaintedSubstitution(target, 'var(--tainted)', { | |
| 28 animationName: 'none', | |
| 29 customProperty: 'tainted', | |
| 30 }); | |
| 31 }, 'Animation tainted values are omitted in CSS property animation-name'); | |
| 32 | |
| 33 test(() => { | |
| 34 var target = createTarget(); | |
| 35 setTaintedValue(target, '--tainted-first', 'tainted'); | |
| 36 target.style.setProperty('--tainted-second', 'var(--tainted-first)'); | |
| 37 testTaintedSubstitution(target, 'var(--tainted-second)', { | |
| 38 animationName: 'none', | |
| 39 customProperty: 'tainted', | |
| 40 }); | |
| 41 }, 'Chained animation tainted values are omitted in CSS property animation-name'
); | |
| 42 | |
| 43 test(() => { | |
| 44 var parent = createTarget(); | |
| 45 var child = parent.appendChild(document.createElement('div')); | |
| 46 setTaintedValue(parent, '--tainted', 'tainted'); | |
| 47 testTaintedSubstitution(child, 'var(--tainted)', { | |
| 48 animationName: 'none', | |
| 49 customProperty: 'tainted', | |
| 50 }); | |
| 51 }, 'Inherited animation tainted values are omitted in CSS property animation-nam
e'); | |
| 52 | |
| 53 test(() => { | |
| 54 var target = createTarget(); | |
| 55 setTaintedValue(target, '--tainted', 'tainted'); | |
| 56 testTaintedSubstitution(target, 'var(--tainted, fallback)', { | |
| 57 animationName: 'fallback', | |
| 58 customProperty: 'tainted', | |
| 59 }); | |
| 60 }, 'Animation tainted values trigger var fallbacks in CSS property animation-nam
e'); | |
| 61 | |
| 62 test(() => { | |
| 63 var target = createTarget(); | |
| 64 setTaintedValue(target, '--tainted', 'tainted'); | |
| 65 testTaintedSubstitution(target, 'var(--unknown, var(--tainted))', { | |
| 66 animationName: 'none', | |
| 67 customProperty: ' tainted', | |
| 68 }); | |
| 69 }, 'Animation tainted fallback values are omitted in CSS property animation-name
'); | |
| 70 </script> | |
| OLD | NEW |
|---|
Chromium Code Reviews
Issue 2521103004:
Revert of Apply custom property animation (Closed)
