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
Fix panic when perspective property is zero. #9446
Conversation
glennw
commented
Jan 28, 2016
r? @pcwalton |
@@ -0,0 +1,30 @@ | |||
<!DOCTYPE html> | |||
<html> | |||
<head> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing <link>
to identify the reference file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated - although it was definitely running that test locally without that line. Just out of curiosity, why do we need to list it both there and also (twice) in the manifest file?
d097981
to
183c3f5
Compare
The manifest file is generated automatically by extracting the metadata from the test files. Hand-editing MANIFEST.json is an anti-pattern; it should be updated via |
@bors-servo: r+ |
📌 Commit 183c3f5 has been approved by |
Fix panic when perspective property is zero. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.svg" height="40" alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9446) <!-- Reviewable:end -->
💔 Test failed - mac-rel-wpt |
@bors-servo retry (Seems infrastructure related - very large number of unrelated tests failed, which also has occurred to several other builds on this machine today). |
⚡ Previous build results for android, gonk, linux-dev, linux-rel, mac-dev-unit, mac-rel-css are reusable. Rebuilding only mac-rel-wpt... |
☀️ Test successful - android, gonk, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt |