Re: RFR: 8240969: WebView does not allow to load style sheet in modularized applications [v4]

2020-08-14 Thread Arun Joseph
On Fri, 14 Aug 2020 06:23:18 GMT, Tobias Diez wrote: >> Currently, loading a style sheet file using >> `WebView.getEngine().setUserStyleSheetLocation(url)` fails if the url >> start's with `jrt`, i.e. if the file is packaged in an application image >> using jlink. This is fixed with this PR.

Integrated: 8240969: WebView does not allow to load style sheet in modularized applications

2020-08-14 Thread Tobias Diez
On Thu, 24 Oct 2019 16:42:14 GMT, Tobias Diez wrote: > Currently, loading a style sheet file using > `WebView.getEngine().setUserStyleSheetLocation(url)` fails if the url > start's with `jrt`, i.e. if the file is packaged in an application image > using jlink. This is fixed with this PR.

Re: RFR: 8240969: WebView does not allow to load style sheet in modularized applications [v3]

2020-08-14 Thread Tobias Diez
On Thu, 13 Aug 2020 21:36:52 GMT, Kevin Rushforth wrote: >> Tobias Diez has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Remove whitespace > > modules/javafx.web/src/test/java/test/javafx/scene/web/MiscellaneousTest.java > line 478: >

Re: RFR: 8240969: WebView does not allow to load style sheet in modularized applications [v4]

2020-08-14 Thread Tobias Diez
> Currently, loading a style sheet file using > `WebView.getEngine().setUserStyleSheetLocation(url)` fails if the url > start's with `jrt`, i.e. if the file is packaged in an application image > using jlink. This is fixed with this PR. Tobias Diez has updated the pull request incrementally with

Re: RFR: 8241840: Memoryleak: Closed focused Stages are not collected with Monocle. [v7]

2020-08-14 Thread Florian Kirmaier
> Closed focused Stages are not collected with Monocle > > This commit adds a unit-test and a fix for collecting focused closed stages. > > ticket: https://bugs.openjdk.java.net/browse/JDK-8241840 Florian Kirmaier has updated the pull request incrementally with one additional commit since the

Re: RFR: 8241840: Memoryleak: Closed focused Stages are not collected with Monocle. [v5]

2020-08-14 Thread Florian Kirmaier
On Thu, 13 Aug 2020 19:31:57 GMT, Kevin Rushforth wrote: >> They are called from the following static method: >> @BeforeClass >> public static void initFX() throws Exception { >> initFXBase(); >> } > > No, these two are only used from `testClosedFocusedStageLeakBase`, which

Re: RFR: 8241840: Memoryleak: Closed focused Stages are not collected with Monocle. [v5]

2020-08-14 Thread Florian Kirmaier
On Thu, 13 Aug 2020 19:49:45 GMT, Kevin Rushforth wrote: >> The fix looks good now. I'll file a follow-up issue for the potential leak >> on the other platforms. >> >> My thinking is that if `focusedWindow` is only used in Monocle, we should >> push the tracking down to the platform rather >>

Re: RFR: 8241840: Memoryleak: Closed focused Stages are not collected with Monocle. [v5]

2020-08-14 Thread Florian Kirmaier
On Thu, 13 Aug 2020 19:26:47 GMT, Kevin Rushforth wrote: >> tests/system/src/test/java/test/javafx/stage/FocusedWindowMonocleTest.java >> line 56: >> >>> 55: } >>> 56: } >> >> Minor: Add missing newline > > I meant that the file should end with a newline (it doesn't matter whether or >

Re: RFR: 8241840: Memoryleak: Closed focused Stages are not collected with Monocle. [v7]

2020-08-14 Thread Ambarish Rapte
On Fri, 14 Aug 2020 10:09:22 GMT, Florian Kirmaier wrote: >> Closed focused Stages are not collected with Monocle >> >> This commit adds a unit-test and a fix for collecting focused closed stages. >> >> ticket: https://bugs.openjdk.java.net/browse/JDK-8241840 > > Florian Kirmaier has updated

Re: RFR: 8240969: WebView does not allow to load style sheet in modularized applications [v4]

2020-08-14 Thread Kevin Rushforth
On Fri, 14 Aug 2020 12:35:00 GMT, Kevin Rushforth wrote: >> Tobias Diez has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Update MiscellaneousTest.java > > Given the simplicity of the fix, I think the minimal test is enough. @arun-Joseph

Re: RFR: 8240969: WebView does not allow to load style sheet in modularized applications [v4]

2020-08-14 Thread Kevin Rushforth
On Fri, 14 Aug 2020 06:23:18 GMT, Tobias Diez wrote: >> Currently, loading a style sheet file using >> `WebView.getEngine().setUserStyleSheetLocation(url)` fails if the url >> start's with `jrt`, i.e. if the file is packaged in an application image >> using jlink. This is fixed with this PR.