On Mon, 20 Jan 2020 16:39:15 GMT, Guru Hb <g...@openjdk.org> wrote:

> Updated GTK Webkit 2.26 (609.1) into jfx.

A sanity check of the patch looks good to me: I did a build / test on all three 
platforms. I'll do a bit more testing, and also review the JavaFX-specific 
files, before completing my review.

To the latter point, I took the patch, applied it to my personal fork, and then 
split it into two commits, one of which has the changes to the files that I 
wanted to take a closer look at (finding just those files in the original PR 
when there are over 5,000 files would be impractical -- impossible given 
GitHub's 3000 file limitation in the file diff veiwer).

In case this is useful to other reviewers, [click 
here](https://github.com/kevinrushforth/jfx/compare/fa6df1ecacc1334b256ffd96dd15cd7dfb1fc108...83d114aed6f9f30ebd90a567192537f62328ffa5)
 for the diffs. It only includes 87 files, many of which are cmake files for 
platforms other than `Java` so don't really need to be reviewed.

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

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

Reply via email to