RFR: 8267554: Support loading stylesheets from data-URIs [v4]

Kevin Rushforth kcr at openjdk.java.net
Wed Jun 23 13:00:31 UTC 2021


On Tue, 22 Jun 2021 23:15:55 GMT, Michael Strauß <mstrauss at openjdk.org> wrote:

>> This PR extends data URI support to allow stylesheets to be loaded from data URIs.
>
> Michael Strauß has updated the pull request incrementally with one additional commit since the last revision:
> 
>   simplified branch in StyleManager.loadStylesheetUnPrivileged

The API and spec changes look good. As I mentioned in a comment added to the CSR, it is ready to move to "Proposed". Once we are farther along in the code review, I'll formally review the CSR.

One thing I noticed is that the diffs don't exactly match what you pasted in the CSR (looks like a copy/paste issue). The `Scene::setUserAgentStylesheet` and `SubScene::setUserAgentStylesheet` methods each have three unchanged lines that are marked with the `+` diff indicator. Can you double-check?

I'll review the implementation next.

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

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


More information about the openjfx-dev mailing list