Chromium Code Reviews Chromium Code Reviews
Issue 2137293003:
Add metadata and improve descriptions for null keyframe offset test. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| OLD | NEW |
|---|---|
| 1 <!DOCTYPE html> | 1 <!DOCTYPE html> |
| 2 <meta charset=utf-8> | |
| 3 <title>Test keyframes with null offsets</title> | |
| 4 <link rel="help" href="https://w3c.github.io/web-animations/#keyframes-section" > | |
| 5 | |
| 2 <script src="../../resources/testharness.js"></script> | 6 <script src="../../resources/testharness.js"></script> |
| 3 <script src="../../resources/testharnessreport.js"></script> | 7 <script src="../../resources/testharnessreport.js"></script> |
| 4 <script src="resources/keyframes-test.js"></script> | 8 <script src="resources/keyframes-test.js"></script> |
| 5 <script> | 9 <script> |
| 6 test(function() { | 10 test(function() { |
| 7 assertAnimationStyles([ | 11 assertAnimationStyles([ |
| 8 {opacity: '0.5', left: '50px'}, | 12 {opacity: '0.5', left: '50px'}, |
| 9 {opacity: '0', left: '0px', offset: null}, | 13 {opacity: '0', left: '0px', offset: null}, |
| 10 {opacity: '0.75', left: '75px', offset: null}, | 14 {opacity: '0.75', left: '75px', offset: null}, |
| 11 ], { | 15 ], { |
| 12 0: {opacity: '0.5', left: '50px'}, | 16 0: {opacity: '0.5', left: '50px'}, |
| 13 0.25: {opacity: '0.25', left: '25px'}, | 17 0.25: {opacity: '0.25', left: '25px'}, |
| 14 0.5: {opacity: '0', left: '0px'}, | 18 0.5: {opacity: '0', left: '0px'}, |
| 15 0.75: {opacity: '0.375', left: '37.5px'}, | 19 0.75: {opacity: '0.375', left: '37.5px'}, |
| 16 1: {opacity: '0.75', left: '75px'}, | 20 1: {opacity: '0.75', left: '75px'}, |
| 17 }, 'case A'); | 21 }, 'Null and missing offsets behave identically'); |
| 18 assertAnimationStyles([ | 22 assertAnimationStyles([ |
| 19 {opacity: '0.5', left: '50px', offset: null}, | 23 {opacity: '0.5', left: '50px', offset: null}, |
| 20 {opacity: '0', left: '0px', offset: 0.25}, | 24 {opacity: '0', left: '0px', offset: 0.25}, |
| 21 {opacity: '0.75', left: '75px', offset: null}, | 25 {opacity: '0.75', left: '75px', offset: null}, |
| 22 ], { | 26 ], { |
| 23 0: {opacity: '0.5', left: '50px'}, | 27 0: {opacity: '0.5', left: '50px'}, |
| 24 0.125: {opacity: '0.25', left: '25px'}, | 28 0.125: {opacity: '0.25', left: '25px'}, |
| 25 0.25: {opacity: '0', left: '0px'}, | 29 0.25: {opacity: '0', left: '0px'}, |
| 26 0.5: {opacity: '0.25', left: '25px'}, | 30 0.5: {opacity: '0.25', left: '25px'}, |
| 27 0.75: {opacity: '0.5', left: '50px'}, | 31 0.75: {opacity: '0.5', left: '50px'}, |
| 28 1: {opacity: '0.75', left: '75px'}, | 32 1: {opacity: '0.75', left: '75px'}, |
| 29 }, 'case B'); | 33 }, 'Null and explicit offsets behave as with missing and explicit offsets.'); |
| 30 }, | 34 }, |
| 31 'element.animate() with null offsets specified', | 35 'element.animate() with null offsets specified', |
| 32 { | 36 { |
| 33 help: 'http://dev.w3.org/fxtf/web-animations/#the-keyframe-dictionary', | 37 help: 'http://dev.w3.org/fxtf/web-animations/#the-keyframe-dictionary', |
| 34 assert: [ | 38 assert: [ |
| 35 'element.animate() should start an animation when keyframes are specified wi th', | 39 'element.animate() should start an animation when keyframes are specified wi th', |
| 36 'null offsets. The behaviour should be identical to not specifying the offse ts.', | 40 'null offsets. The behaviour should be identical to not specifying the offse ts.', |
| 37 ], | 41 ], |
| 38 author: 'Alan Cutter', | 42 author: 'Alan Cutter', |
|
alancutter (OOO until 2018)
2016/07/12 05:38:38
Remove this third parameter.
| |
| 39 }); | 43 }); |
| 40 </script> | 44 </script> |
| OLD | NEW |
