On Fri, 7 Aug 2020 14:54:14 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
>> Bhawesh Choudhary has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Removed RenderSVGResourceMasker changes and added fix for HIDPI mask
>> rendering
>
> modules/javafx.web/src/main/native/Source/WebCore/platform/graphics/java/GraphicsContextJava.cpp
> line 235:
>
>> 233: } else {
>> 234: if (m_state.fillGradient) {
>> 235: setCTM(m_state.transform);
>
> Why is this needed here, but not in the other places `setGradient` is called?
> Won't there be a similar problem with
> `strokeRect`, `fillPath`, etc?
This question is still outstanding. It seems like the call to `setCTM` is
either needed before all of the `setGradient`
calls or none of them. Can you comment?
-------------
PR: https://git.openjdk.java.net/jfx/pull/213
