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

[Tobias Diez]

On Thu, 13 Aug 2020 21:36:52 GMT, Kevin Rushforth <kcr at openjdk.org> 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:
> 
>> 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.

Thanks for the feedback.

How is `tests/manual` invoked? I don't see how it would make it possible to load the jrt url successful (as you need to
have a modularized app for this, or not?). Moreover, the test here was supposed to only check that jrt url's are no
longer rejected. Not that the css is successfully applied (since there is no bug in this part of the code, and I
couldn't find such a test for the loading of a normal "file://" or "jar://" either).

-------------

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