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

Gradient reftest regression #1051

Closed
glennw opened this issue Apr 3, 2017 · 2 comments
Closed

Gradient reftest regression #1051

glennw opened this issue Apr 3, 2017 · 2 comments

Comments

@glennw
Copy link
Member

@glennw glennw commented Apr 3, 2017

The following Servo tests are broken by #1046.

  ▶ FAIL [expected PASS] /_mozilla/css/border-image-linear-gradient.html
  └   → /_mozilla/css/border-image-linear-gradient.html e68bebc2dbcc53b852fd3b6e77485a9031fb5f35
/_mozilla/css/border-image-linear-gradient-ref.html 57d716ccc54e3f76d38f5babca226e3878731ca3
Testing e68bebc2dbcc53b852fd3b6e77485a9031fb5f35 == 57d716ccc54e3f76d38f5babca226e3878731ca3

  ▶ FAIL [expected PASS] /_mozilla/css/linear_gradients_parsing_a.html
  └   → /_mozilla/css/linear_gradients_parsing_a.html 95f9d6467048d6325dd24fe511c046a8a77d050e
/_mozilla/css/linear_gradients_parsing_ref.html 943c7b8a33e43d322e511d8ec6784868b781ac75
Testing 95f9d6467048d6325dd24fe511c046a8a77d050e == 943c7b8a33e43d322e511d8ec6784868b781ac75

We have another change landed in WR that I'd like to get into a WR update for Servo (may help with some of our intermittents). Is this likely to be a relatively simple fix, or should we revert for now?

cc @rlhunt @kvark

@eqrion
Copy link
Contributor

@eqrion eqrion commented Apr 3, 2017

I will take a look at this as soon as I can.

@eqrion
Copy link
Contributor

@eqrion eqrion commented Apr 3, 2017

The issue here is a few things:

  1. One of the patches added a condition to when we use aligned gradients, but it is flipped backwards causing angle gradients to always be used
  2. Angle gradients still have dithering enabled, even though aligned gradients had them disabled
  3. There are reftest issues with dithering (should be fixed by #1045)

I will add a PR fixing the aligned gradient condition which fixed the issue for me.

bors-servo added a commit that referenced this issue Apr 3, 2017
Fix aligned gradient condition

Fixes #1051

<!-- 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/1057)
<!-- Reviewable:end -->
bors-servo added a commit that referenced this issue Apr 3, 2017
Fix aligned gradient condition

Fixes #1051

<!-- 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/1057)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.