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

add reftest for color emoji fast shadows #2233

Merged
merged 1 commit into from Dec 19, 2017

Conversation

@lsalzman
Copy link
Contributor

lsalzman commented Dec 18, 2017

This is a follow-up to #2223. It ensures that the color bitmap's color does not improperly leak into the shadow. It is Mac only for now due to the relative impossibility of making cross-platform color bitmap fonts.


This change is Reviewable

@glennw
Copy link
Member

glennw commented Dec 18, 2017

Thanks!

@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented Dec 18, 2017

📌 Commit 1e2ab6c has been approved by glennw

@bors-servo
Copy link
Contributor

bors-servo commented Dec 18, 2017

Testing commit 1e2ab6c with merge 8b97c7c...

bors-servo added a commit that referenced this pull request Dec 18, 2017
add reftest for color emoji fast shadows

This is a follow-up to #2223. It ensures that the color bitmap's color does not improperly leak into the shadow. It is Mac only for now due to the relative impossibility of making cross-platform color bitmap fonts.

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

bors-servo commented Dec 18, 2017

💔 Test failed - status-travis

@glennw
Copy link
Member

glennw commented Dec 18, 2017

@bors-servo retry delegate+

@bors-servo
Copy link
Contributor

bors-servo commented Dec 18, 2017

✌️ @lsalzman can now approve this pull request

@bors-servo
Copy link
Contributor

bors-servo commented Dec 18, 2017

Testing commit 1e2ab6c with merge 1142dfc...

bors-servo added a commit that referenced this pull request Dec 18, 2017
add reftest for color emoji fast shadows

This is a follow-up to #2223. It ensures that the color bitmap's color does not improperly leak into the shadow. It is Mac only for now due to the relative impossibility of making cross-platform color bitmap fonts.

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

bors-servo commented Dec 19, 2017

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

@bors-servo bors-servo merged commit 1e2ab6c into servo:master Dec 19, 2017
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

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