New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CanvasGradient#addColorStop should throw for invalid colors and offsets #7104
Conversation
@bors-servo r+ -S-awaiting-review +S-awaiting-merge Reviewed 5 of 5 files at r1. Comments from the review on Reviewable.io |
📌 Commit 33ccb64 has been approved by |
CanvasGradient#addColorStop should throw for invalid colors and offsets The new test failure is because the color stop has a value of 'currentColor' which we don't support yet. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7104) <!-- Reviewable:end -->
💔 Test failed - linux2 |
@bors-servo r-
|
@Ms2ger Oh! |
33ccb64
to
f41aee1
Compare
f41aee1
to
7315b50
Compare
@bors-servo r+ |
📌 Commit 7315b50 has been approved by |
CanvasGradient#addColorStop should throw for invalid colors and offsets The new test failure is because the color stop has a value of 'currentColor' which we don't support yet. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7104) <!-- Reviewable:end -->
The new test failure is because the color stop has a value of 'currentColor' which we don't support yet.