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

Round blur support up to the nearest even number of pixels. #3126

Merged
merged 1 commit into from Sep 27, 2018

Conversation

@pcwalton
Copy link
Collaborator

pcwalton commented Sep 26, 2018

I'm pretty confident this is correct--it's either just as accurate or more
accurate than the current implementation--so it'll let me sort out which test failures, if any, from #3028 and #3122 are true regressions.


This change is Reviewable

@kvark
kvark approved these changes Sep 26, 2018
@pcwalton
Copy link
Collaborator Author

pcwalton commented Sep 26, 2018

How do I mark various failing Gecko tests as fuzzy?

@mstange
Copy link
Contributor

mstange commented Sep 26, 2018

With a Gecko patch that modifies the reftest.list files in the directories of the tests.

@pcwalton
Copy link
Collaborator Author

pcwalton commented Sep 26, 2018

@mstange Does that need to land before or after the corresponding patch lands in the WR repository?

@gw3583
Copy link
Collaborator

gw3583 commented Sep 26, 2018

I generally get @jrmuizel or @staktrace to sign off on the fuzziness increases being OK here in a comment, and then they update them as part of the WR update.

@mstange
Copy link
Contributor

mstange commented Sep 26, 2018

It needs to land together with the WebRender update that pulls that change into Gecko. So just attach the patch to the wr-future-update bug and Jeff or Kats will include it.

@pcwalton
Copy link
Collaborator Author

pcwalton commented Sep 26, 2018

Gotcha, thanks.

I'm pretty confident this is correct--it's either just as accurate or more
accurate than the current implementation--so it'll let me sort out which test
failures, if any, from #3028 and #3122 are true regressions.
@pcwalton pcwalton force-pushed the pcwalton:blur-even-support branch from b1b1f81 to 61d43da Sep 26, 2018
@pcwalton
Copy link
Collaborator Author

pcwalton commented Sep 26, 2018

I verified all test differences are imperceptible. Will carry forward review once tests pass unless anyone has objections. I will also submit a patch to fix Gecko test suite failures.

@pcwalton
Copy link
Collaborator Author

pcwalton commented Sep 26, 2018

@bors-servo: r=kvark

@bors-servo
Copy link
Contributor

bors-servo commented Sep 26, 2018

📌 Commit 61d43da has been approved by kvark

@bors-servo
Copy link
Contributor

bors-servo commented Sep 26, 2018

Testing commit 61d43da with merge 0d2e961...

bors-servo added a commit that referenced this pull request Sep 26, 2018
Round blur support up to the nearest even number of pixels.

I'm pretty confident this is correct--it's either just as accurate or more
accurate than the current implementation--so it'll let me sort out which test failures, if any, from #3028 and #3122 are true regressions.

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

bors-servo commented Sep 27, 2018

☀️ Test successful - status-appveyor, status-taskcluster
Approved by: kvark
Pushing 0d2e961 to master...

@bors-servo bors-servo merged commit 61d43da into servo:master Sep 27, 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
@pcwalton pcwalton deleted the pcwalton:blur-even-support branch Sep 27, 2018
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

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