Re: RFR: JDK-8310681: Update WebKit to 616.1 [v2]

2023-07-28 Thread Joeri Sykora
On Mon, 24 Jul 2023 12:43:57 GMT, Hima Bindu Meda wrote: >> Updated JavaFX Webkit to GTK WebKit 2.40 (616.1). >> Verified the updated version build, sanity tests and stability. No issues >> have been observed. > > Hima Bindu Meda has updated the pull request incrementally with one > additional

Re: RFR: JDK-8310681: Update WebKit to 616.1 [v2]

2023-07-24 Thread Kevin Rushforth
On Mon, 24 Jul 2023 12:43:57 GMT, Hima Bindu Meda wrote: >> Updated JavaFX Webkit to GTK WebKit 2.40 (616.1). >> Verified the updated version build, sanity tests and stability. No issues >> have been observed. > > Hima Bindu Meda has updated the pull request incrementally with one > additional

Re: RFR: JDK-8310681: Update WebKit to 616.1

2023-07-24 Thread Hima Bindu Meda
On Mon, 24 Jul 2023 02:50:30 GMT, Jay Bhaskar wrote: > > Oh, I see that now. What led me to ask this question was that there is an > > alternative implementation in `#if PLATFORM(JAVA)`, which implies that we > > removed the call to `stripLeadingAndTrailingHTMLSpaces` and that it is > > still

Re: RFR: JDK-8310681: Update WebKit to 616.1 [v2]

2023-07-24 Thread Hima Bindu Meda
> Updated JavaFX Webkit to GTK WebKit 2.40 (616.1). > Verified the updated version build, sanity tests and stability. No issues > have been observed. Hima Bindu Meda has updated the pull request incrementally with one additional commit since the last revision: Remove check for PLATFORM(JAVA)

Re: RFR: JDK-8310681: Update WebKit to 616.1

2023-07-23 Thread Jay Bhaskar
On Sun, 23 Jul 2023 15:55:39 GMT, Johan Vos wrote: > > Oh, I see that now. What led me to ask this question was that there is an > > alternative implementation in `#if PLATFORM(JAVA)`, which implies that we > > removed the call to `stripLeadingAndTrailingHTMLSpaces` and that it is > > still pr

Re: RFR: JDK-8310681: Update WebKit to 616.1

2023-07-23 Thread Jay Bhaskar
On Fri, 21 Jul 2023 16:52:27 GMT, Jay Bhaskar wrote: > absoluteLinkURL Agree - PR Comment: https://git.openjdk.org/jfx/pull/1180#issuecomment-1647113499

Re: RFR: JDK-8310681: Update WebKit to 616.1

2023-07-23 Thread Jay Bhaskar
On Sun, 23 Jul 2023 10:28:18 GMT, yosbits wrote: > I found a change to turn off DFGJIT by default, is this a fix for some > problem? > > modules/javafx.web/src/main/java/com/sun/webkit/WebPage.java The Change is required, as it is making issues with complex js code written for a website. ---

Re: RFR: JDK-8310681: Update WebKit to 616.1

2023-07-23 Thread Johan Vos
On Fri, 21 Jul 2023 17:04:44 GMT, Kevin Rushforth wrote: > Oh, I see that now. What led me to ask this question was that there is an > alternative implementation in `#if PLATFORM(JAVA)`, which implies that we > removed the call to `stripLeadingAndTrailingHTMLSpaces` and that it is still > pres

Re: RFR: JDK-8310681: Update WebKit to 616.1

2023-07-23 Thread yosbits
On Thu, 20 Jul 2023 15:36:28 GMT, Hima Bindu Meda wrote: > Updated JavaFX Webkit to GTK WebKit 2.40 (616.1). > Verified the updated version build, sanity tests and stability. No issues > have been observed. I found a change to turn off DFGJIT by default, is this a fix for some problem? modules

Re: RFR: JDK-8310681: Update WebKit to 616.1

2023-07-21 Thread Kevin Rushforth
On Thu, 20 Jul 2023 15:36:28 GMT, Hima Bindu Meda wrote: > Updated JavaFX Webkit to GTK WebKit 2.40 (616.1). > Verified the updated version build, sanity tests and stability. No issues > have been observed. @johanvos or @tiainen Can one you be the second reviewer? - PR Comment: ht

Re: RFR: JDK-8310681: Update WebKit to 616.1

2023-07-21 Thread Kevin Rushforth
On Thu, 20 Jul 2023 15:36:28 GMT, Hima Bindu Meda wrote: > Updated JavaFX Webkit to GTK WebKit 2.40 (616.1). > Verified the updated version build, sanity tests and stability. No issues > have been observed. Looks good. Testing is green. - Marked as reviewed by kcr (Lead). PR Revi

Re: RFR: JDK-8310681: Update WebKit to 616.1

2023-07-21 Thread Hima Bindu Meda
On Thu, 20 Jul 2023 15:36:28 GMT, Hima Bindu Meda wrote: > Updated JavaFX Webkit to GTK WebKit 2.40 (616.1). > Verified the updated version build, sanity tests and stability. No issues > have been observed. > You can include this in the next "cherry-pick" update, if you prefer. > You can inc

Re: RFR: JDK-8310681: Update WebKit to 616.1

2023-07-21 Thread Kevin Rushforth
On Thu, 20 Jul 2023 15:36:28 GMT, Hima Bindu Meda wrote: > Updated JavaFX Webkit to GTK WebKit 2.40 (616.1). > Verified the updated version build, sanity tests and stability. No issues > have been observed. You can include this in the next "cherry-pick" update, if you prefer. - PR

Re: RFR: JDK-8310681: Update WebKit to 616.1

2023-07-21 Thread Kevin Rushforth
On Thu, 20 Jul 2023 15:36:28 GMT, Hima Bindu Meda wrote: > Updated JavaFX Webkit to GTK WebKit 2.40 (616.1). > Verified the updated version build, sanity tests and stability. No issues > have been observed. Oh, I see that now. What led me to ask this question was that there is an alternative i

Re: RFR: JDK-8310681: Update WebKit to 616.1

2023-07-21 Thread Jay Bhaskar
On Fri, 21 Jul 2023 14:24:17 GMT, Kevin Rushforth wrote: > Is this new code or was it moved / refactored from elsewhere? If new code, > what is its purpose, and who is calling it? It was taken from last webkit update, as it was giving undefined reference error for current webkit update. > ```

Re: RFR: JDK-8310681: Update WebKit to 616.1

2023-07-21 Thread Kevin Rushforth
On Thu, 20 Jul 2023 15:36:28 GMT, Hima Bindu Meda wrote: > Updated JavaFX Webkit to GTK WebKit 2.40 (616.1). > Verified the updated version build, sanity tests and stability. No issues > have been observed. I have two questions. Because of the large number of files, I can't use the "diffs" vie

RFR: JDK-8310681: Update WebKit to 616.1

2023-07-20 Thread Hima Bindu Meda
Updated JavaFX Webkit to GTK WebKit 2.40 (616.1). Verified the updated version build, sanity tests and stability. No issues have been observed. - Commit messages: - Update WebKit to 616.1 Changes: https://git.openjdk.org/jfx/pull/1180/files Webrev: https://webrevs.openjdk.org/?rep