Skip to content
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 perspective-origin reftest. #2398

Merged
merged 1 commit into from Feb 8, 2018
Merged

Fix perspective-origin reftest. #2398

merged 1 commit into from Feb 8, 2018

Conversation

@glennw
Copy link
Member

glennw commented Feb 8, 2018

I was misinterpreting the output of this test - I hadn't realized
that in wrench the stretch-size defaults to the image dimensions,
rather than the local rect dimensions. I modified the test to
take account of this and it draws correctly now.

Fixes #2379.


This change is Reviewable

I was misinterpreting the output of this test - I hadn't realized
that in wrench the stretch-size defaults to the image dimensions,
rather than the local rect dimensions. I modified the test to
take account of this and it draws correctly now.

Fixes #2379.
@glennw
Copy link
Member Author

glennw commented Feb 8, 2018

r? @kvark or anyone

I temporarily removed the fuzziness from this test, to see what CI fails with. It passes locally on my machine with no fuzziness now. We might still need a fuzz setting on CI, but it will hopefully be less.

@kvark
kvark approved these changes Feb 8, 2018
@kvark
Copy link
Member

kvark commented Feb 8, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Feb 8, 2018

📌 Commit 3a15e33 has been approved by kvark

@bors-servo
Copy link
Contributor

bors-servo commented Feb 8, 2018

Testing commit 3a15e33 with merge 342bc31...

bors-servo added a commit that referenced this pull request Feb 8, 2018
Fix perspective-origin reftest.

I was misinterpreting the output of this test - I hadn't realized
that in wrench the stretch-size defaults to the image dimensions,
rather than the local rect dimensions. I modified the test to
take account of this and it draws correctly now.

Fixes #2379.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/2398)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Feb 8, 2018

☀️ Test successful - status-appveyor, status-taskcluster, status-travis
Approved by: kvark
Pushing 342bc31 to master...

@bors-servo bors-servo merged commit 3a15e33 into servo:master Feb 8, 2018
3 checks passed
3 checks passed
Taskcluster (pull_request) TaskGroup: success
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.