[webkit] [9] Review request for 8170450: Crash while loading wordpress.com in HiDPI / Retina display

Hello Kevin, Guru, Murali, Please review the fix for JDK-8170450 . Analysis: While applying a filter, WebCore get's ImageData from ImageDataJava using getPremultipliedImageData method. Our current implementation doesn't considers resolutionScal

[9] Review request for 8151170: WINDOW_MODAL dialog does not popup when restoring iconified owner window, which causes application freeze

Hello Kevin & David, Please review the fix for jfx9: bug: https://bugs.openjdk.java.net/browse/JDK-8151170 webrev: http://cr.openjdk.java.net/~ssadetsky/8151170/webrev.00/ Win32 API ::GetWindowRect() call returns fake -32000,-32000 location for iconified window. Solution: skip WM_MOVE if wind

JDK-8169649 Review request

Hello, everyone. Please have a look at my review request for new small test: Jira issue: https://bugs.openjdk.java.net/browse/JDK-8169649 Webrev: http://cr.openjdk.java.net/~arusakov/8169649/webrev.00/ Thanks in advance.

Re: Follow-on bugs

Jim, As announced yesterday night, I made a new 'Double' (alias D) pipeline for marlinFX with 2 webrevs: - cmp that compares the D pipeline vs the float pipeline: http://cr.openjdk.java.net/~lbourges/marlinFX-D/marlinFX-Double-cmp/ - raw that makes no comparison to be easily applicable as a patch:

Re: Follow-on bugs

Jim, I found the bug in RendererNoAA caused by a 'bad' regexp match: 0x7fff was modified to 0x7Dff ! so there is no more regression in TestNonAARasterization ! Laurent FYI, here is the diff of the correction (over the previous big patches): < +++ new/modules/javafx.graphics/src/main/j

Re: Follow-on bugs

One thing that might cause a problem is that the script modified ERR_STEP_MAX because it had 7f, but it was 0x7f, not 1.7f. Some other constants might be affected as well... ...jim On 11/30/16 2:12 PM, Laurent Bourgès wrote: - TestNonAARasterization: the errors seem ar

Re: Follow-on bugs

If this eliminates the regressions in TestNonAA, then I'm for reworking this as an in-place patch for MarlinFX. We can't use the existing Dasher bug for this because Marlin isn't the default renderer, but we can mention the DMarlinFX bug as a workaround for the Dasher bug...