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

Sync changes from mozilla-central gfx/wr #3839

Merged
merged 2 commits into from Jan 22, 2020
Merged

Conversation

@moz-gfx
Copy link

moz-gfx commented Jan 22, 2020

No description provided.

gw3583 added 2 commits Jan 22, 2020
… in some cases. r=Bert

The semantics of the inaccuracy reftest mode are that the overall
test succeeds as long as one of the multiple test images mismatches
the reference image.

However, the previous implementation would print an unexpected result
if any of the comparisons were equal (even though the overall test
result would be reported correctly).

This patch changes wrench so that it only reports an unexpected
failure for the overall test, not each individual reference.

Differential Revision: https://phabricator.services.mozilla.com/D60564

[ghsync] From https://hg.mozilla.org/mozilla-central/rev/3d076ae68a9411e4f412c90d2743987f7b916850
…taro,nical

There are a number of issues with the current gradient dithering
implementation, that cause many test failures and also fuzziness
rendering when enabling DirectComposition virtual surfaces. In
particular, the dither result is dependent on the offset of the
update rect within a render target.

For now, this patch disables gradient dithering by default. This
gives us:

 - A heap of new test PASS results (or reduced fuzziness).
 - Fixes a number of non-deterministic fuzziness bugs with DC.
 - Improves performance of gradient rendering by a reasonable amount.

We can fix gradient dithering as a follow up, and re-enable if/when
we find content that would benefit from it significantly (we may
be able to improve gradients in other ways than dithering too).

Differential Revision: https://phabricator.services.mozilla.com/D60460

[ghsync] From https://hg.mozilla.org/mozilla-central/rev/f758f235dd70f1060dba771ccfd3cb91a9aaef20
@moz-gfx
Copy link
Author

moz-gfx commented Jan 22, 2020

@bors-servo r=auto

@bors-servo
Copy link
Contributor

bors-servo commented Jan 22, 2020

📌 Commit 580d7b2 has been approved by auto

@bors-servo
Copy link
Contributor

bors-servo commented Jan 22, 2020

Testing commit 580d7b2 with merge 6e9a9aa...

bors-servo added a commit that referenced this pull request Jan 22, 2020
Sync changes from mozilla-central gfx/wr
@bors-servo
Copy link
Contributor

bors-servo commented Jan 22, 2020

☀️ Test successful - status-appveyor, status-taskcluster
Approved by: auto
Pushing 6e9a9aa to master...

@bors-servo bors-servo merged commit 580d7b2 into servo:master Jan 22, 2020
3 checks passed
3 checks passed
Community-TC (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

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