Re: RFR: 8262276: Debug build of WebKit fails [v3]

2021-04-23 Thread Johan Vos
On Tue, 20 Apr 2021 20:03:28 GMT, Arun Joseph  wrote:

>> Fixing the Debug build of WebKit.
>> 
>> Test: Build JavaFX using `-PCOMPILE_WEBKIT=true -PCONF=DebugNative` and test 
>> using a simple HelloWebView app.
>
> Arun Joseph has updated the pull request with a new target base due to a 
> merge or a rebase. The pull request now contains three commits:
> 
>  - Merge
>  - Fix InternalFunction crash
>  - 8262276: Debug build of WebKit fails

I can't reproduce my previous issues, so I assume there was something wrong 
with my local environment (which is likely).
Looks good now.

-

Marked as reviewed by jvos (Reviewer).

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


Re: RFR: 8262276: Debug build of WebKit fails [v3]

2021-04-23 Thread Johan Vos
On Tue, 20 Apr 2021 20:03:28 GMT, Arun Joseph  wrote:

>> Fixing the Debug build of WebKit.
>> 
>> Test: Build JavaFX using `-PCOMPILE_WEBKIT=true -PCONF=DebugNative` and test 
>> using a simple HelloWebView app.
>
> Arun Joseph has updated the pull request with a new target base due to a 
> merge or a rebase. The pull request now contains three commits:
> 
>  - Merge
>  - Fix InternalFunction crash
>  - 8262276: Debug build of WebKit fails

I'll re-run my tests

-

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


Re: RFR: 8262276: Debug build of WebKit fails [v3]

2021-04-20 Thread Kevin Rushforth
On Tue, 20 Apr 2021 20:03:28 GMT, Arun Joseph  wrote:

>> Fixing the Debug build of WebKit.
>> 
>> Test: Build JavaFX using `-PCOMPILE_WEBKIT=true -PCONF=DebugNative` and test 
>> using a simple HelloWebView app.
>
> Arun Joseph has updated the pull request with a new target base due to a 
> merge or a rebase. The pull request now contains three commits:
> 
>  - Merge
>  - Fix InternalFunction crash
>  - 8262276: Debug build of WebKit fails

Marked as reviewed by kcr (Lead).

-

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


Re: RFR: 8262276: Debug build of WebKit fails [v3]

2021-04-20 Thread Arun Joseph
> Fixing the Debug build of WebKit.
> 
> Test: Build JavaFX using `-PCOMPILE_WEBKIT=true -PCONF=DebugNative` and test 
> using a simple HelloWebView app.

Arun Joseph has updated the pull request with a new target base due to a merge 
or a rebase. The pull request now contains three commits:

 - Merge
 - Fix InternalFunction crash
 - 8262276: Debug build of WebKit fails

-

Changes: https://git.openjdk.java.net/jfx/pull/417/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jfx&pr=417&range=02
  Stats: 34 lines in 6 files changed: 21 ins; 0 del; 13 mod
  Patch: https://git.openjdk.java.net/jfx/pull/417.diff
  Fetch: git fetch https://git.openjdk.java.net/jfx pull/417/head:pull/417

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