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

Reverse apply filters. #2150

Merged
merged 1 commit into from
Dec 4, 2017
Merged

Reverse apply filters. #2150

merged 1 commit into from
Dec 4, 2017

Conversation

mephisto41
Copy link
Contributor

@mephisto41 mephisto41 commented Dec 1, 2017

The order of composite_ops.filters is as same as the order of yaml. But we need reverse apply it to the picture primitive so that the rendering order would match the order of yaml.


This change is Reviewable

@glennw
Copy link
Member

glennw commented Dec 1, 2017

Can we do a gecko try for this?

@mephisto41
Copy link
Contributor Author

@glennw
Copy link
Member

glennw commented Dec 4, 2017

Looks like a few failures (although some of them may be intermittent / unrelated). It'd be good to find out what the cause is though - especially the failures in R7.

@mephisto41
Copy link
Contributor Author

moz-element.html is unxpected-pass. And same-filter.html need change fuzzy values (only add 1 to the max difference value). Others are intermittent. I think it is good to land.

@glennw
Copy link
Member

glennw commented Dec 4, 2017

Sounds good, thanks!

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit c378cf9 has been approved by glennw

@bors-servo
Copy link
Contributor

⌛ Testing commit c378cf9 with merge 5e6494c...

bors-servo pushed a commit that referenced this pull request Dec 4, 2017
Reverse apply filters.

The order of ```composite_ops.filters``` is as same as the order of yaml. But we need reverse apply it to the picture primitive so that the rendering order would match the order of yaml.

<!-- 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/2150)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - status-appveyor, status-travis
Approved by: glennw
Pushing 5e6494c to master...

@bors-servo bors-servo merged commit c378cf9 into servo:master Dec 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants