On Thu, 21 Jan 2021 14:05:54 GMT, Arun Joseph <ajos...@openjdk.org> wrote:

>> I reviewed all of the changes outside `modules/javafx.web/src/main/native/`. 
>> Most look related to the change to use the Prism Color object instead of an 
>> int. I did have a question about one of those changes:
>> 
>> modules/javafx.web/src/main/java/com/sun/webkit/plugin/DefaultPlugin.java
>> -        g.fillRect(x, y, w, h, 0x11aaffff);
>> +        g.fillRect(x, y, w, h, new Color(0.33f, 1.0f, 1.0f, 0.1f));
>> 
>> These are not equivalent colors. Is this intentional?
>
> It should be `new Color(2 / 3.0f, 1.0f, 1.0f, 1 / 15.0f)`. I'll update the PR.

Build and tests pass on windows/linux/mac in our CI.

-------------

PR: https://git.openjdk.java.net/jfx/pull/382

Reply via email to