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
Support all image sampler kinds with brush_image implementation. #2412
Conversation
I haven't tested this in a try build yet - will be able to kick one off in an hour or so. r? @kvark cc @staktrace |
@@ -46,7 +52,7 @@ void brush_vs( | |||
vec4 brush_fs() { | |||
vec2 uv = clamp(vUv.xy, vUvBounds.xy, vUvBounds.zw); | |||
|
|||
vec4 color = texture(sColor0, vec3(uv, vUv.z)); | |||
vec4 color = TEX_SAMPLE(sColor0, vec3(uv, vUv.z)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, this would be incorrect, given that we want to use the mipmaps, and this macro always picks lod of 0.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, fixed.
As a follow up, we need to add better test coverage of these other image types via a rawtest.
5461682
to
85ebb41
Compare
Thanks Glenn! I'd like to see a try push, unless @kats wants to get this in quickly and bear with consequences :) |
At the moment there's a big backlog of windows jobs in gecko CI, so I'm ok with landing this without a try push. If anything turns up I'll let you know. |
@bors-servo r+ |
📌 Commit 85ebb41 has been approved by |
Support all image sampler kinds with brush_image implementation. As a follow up, we need to add better test coverage of these other image types via a rawtest. <!-- 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/2412) <!-- Reviewable:end -->
☀️ Test successful - status-appveyor, status-taskcluster, status-travis |
As a follow up, we need to add better test coverage of these other
image types via a rawtest.
This change is