RFR: 8259635: Update to 610.2 version of WebKit

2021-01-20 Thread Arun Joseph
Update JavaFX WebKit to GTK WebKit 2.30 (610.2) - Commit messages: - 8259635: Update to 610.2 version of WebKit Changes: https://git.openjdk.java.net/jfx/pull/382/files Webrev: https://webrevs.openjdk.java.net/?repo=jfx&pr=382&range=00 Issue: https://bugs.openjdk.java.net/browse/

Re: RFR: 8259635: Update to 610.2 version of WebKit

2021-01-21 Thread Kevin Rushforth
On Thu, 21 Jan 2021 04:14:28 GMT, Arun Joseph wrote: > Update JavaFX WebKit to GTK WebKit 2.30 (610.2) 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 o

Re: RFR: 8259635: Update to 610.2 version of WebKit

2021-01-21 Thread Arun Joseph
On Thu, 21 Jan 2021 13:25:39 GMT, Kevin Rushforth wrote: >> Update JavaFX WebKit to GTK WebKit 2.30 (610.2) > > 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

Re: RFR: 8259635: Update to 610.2 version of WebKit

2021-01-21 Thread Johan Vos
On Thu, 21 Jan 2021 14:05:54 GMT, Arun Joseph 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

Re: RFR: 8259635: Update to 610.2 version of WebKit [v2]

2021-01-21 Thread Arun Joseph
> Update JavaFX WebKit to GTK WebKit 2.30 (610.2) Arun Joseph has updated the pull request incrementally with one additional commit since the last revision: Fix color in DefaultPlugin - Changes: - all: https://git.openjdk.java.net/jfx/pull/382/files - new: https://git.openjdk

Re: RFR: 8259635: Update to 610.2 version of WebKit [v2]

2021-01-21 Thread Kevin Rushforth
On Thu, 21 Jan 2021 15:57:04 GMT, Arun Joseph wrote: >> Update JavaFX WebKit to GTK WebKit 2.30 (610.2) > > Arun Joseph has updated the pull request incrementally with one additional > commit since the last revision: > > Fix color in DefaultPlugin Looks good. Build + automated tests passed o

Re: RFR: 8259635: Update to 610.2 version of WebKit [v2]

2021-01-21 Thread Kevin Rushforth
On Thu, 21 Jan 2021 19:11:21 GMT, Kevin Rushforth wrote: >> Arun Joseph has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Fix color in DefaultPlugin > > Looks good. Build + automated tests passed on all three platforms. I ran into > one m

Re: RFR: 8259635: Update to 610.2 version of WebKit [v2]

2021-01-22 Thread Guru Hb
On Thu, 21 Jan 2021 15:57:04 GMT, Arun Joseph wrote: >> Update JavaFX WebKit to GTK WebKit 2.30 (610.2) > > Arun Joseph has updated the pull request incrementally with one additional > commit since the last revision: > > Fix color in DefaultPlugin Changes looks good to me. Did a local build

Re: RFR: 8259635: Update to 610.2 version of WebKit [v2]

2021-01-29 Thread Johan Vos
On Thu, 21 Jan 2021 15:57:04 GMT, Arun Joseph wrote: >> Update JavaFX WebKit to GTK WebKit 2.30 (610.2) > > Arun Joseph has updated the pull request incrementally with one additional > commit since the last revision: > > Fix color in DefaultPlugin Marked as reviewed by jvos (Reviewer).