Chromium Code Reviews Chromium Code Reviews
Issue 1087283002:
Rework the checks for too-few values in feColorMatrix filter. (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/svg/filters/feColorMatrix-setAttribute-crash1.html |
| diff --git a/LayoutTests/svg/filters/feColorMatrix-setAttribute-crash1.html b/LayoutTests/svg/filters/feColorMatrix-setAttribute-crash1.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..dc280a52aae6226e83dfdfea80d487d2346fb2a4 |
| --- /dev/null |
| +++ b/LayoutTests/svg/filters/feColorMatrix-setAttribute-crash1.html |
| @@ -0,0 +1,39 @@ |
| +<!DOCTYPE html> |
| +<html> |
| +<head> |
| + <script> |
| + function testDone() { |
| + if (window.testRunner) { |
| + testRunner.dumpAsText(); |
| + testRunner.notifyDone(); |
| + } |
| + } |
| + |
| + function changeValues() { |
| + document.getElementById('color').setAttribute('values', '1'); |
| + requestAnimationFrame(testDone); |
| + } |
| + |
| + function forcePaint() { |
| + requestAnimationFrame(changeValues); |
| + } |
| + |
| + onload = function() { |
| + if (window.testRunner) { |
| + testRunner.waitUntilDone(); |
| + } |
| + requestAnimationFrame(forcePaint); |
|
pdr.
2015/04/16 19:19:16
Does this test really need three requestAnimationF
Stephen Chennney
2015/04/16 19:30:32
It's needed. The first one comes before any frame,
|
| + } |
| + </script> |
| +</head> |
| +<body> |
| + <svg id="svg" xmlns="http://www.w3.org/2000/svg" width="100" height="100" filter="url(#MyFilter)" version="1.1"> |
| + <defs> |
| + <filter id="MyFilter"> |
| + <fecolormatrix id="color" type="matrix"></fecolormatrix> |
| + </filter> |
| + </defs> |
| + </svg> |
| + PASS |
| +</body> |
| +</html> |
