On Sun, 2 Aug 2020 10:27:17 GMT, Tobias Diez <github.com+5037600+tobiasd...@openjdk.org> 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. > > Tobias Diez has updated the pull request incrementally with one additional > commit since the last revision: > > Remove whitespace Changes requested by kcr (Lead). modules/javafx.web/src/test/java/test/javafx/scene/web/MiscellaneousTest.java line 478: > 477: @Test public void loadJrtCssFileSuccessfully() { > 478: > getEngine().setUserStyleSheetLocation("jrt:/javafx.web/html/imported-styles.css"); > 479: } This won't run as is for a couple reasons. First it's on the wrong thread, but more importantly, you will not be able to load anything from the unit test using a `jrt` URL, so I don't think a unit test in the `javafx.web` module itself will work. You will either need to provide a system test in `tests/system/` that creates a modular jar file (which is probably overkill for this fix), or else a manual test in `tests/manual`. The latter seems sufficient for this bug and will be easier. ------------- PR: https://git.openjdk.java.net/jfx/pull/22