Skip to main content

Browse the archive

Revision 7d92dbd58ac296da6fb2e24d9c9758775e873088 authored by Mike West on 06 December 2018, 10:19:53 UTC, committed by Chromium WPT Sync on 06 December 2018, 10:19:53 UTC 
Trusted Types: URLs should be USVString, not DOMString.
 
This patch adjusts `ScriptURLString` to be a union including `USVString`,
not `DOMString`. The advice in [WebIDL][1] isn't exactly clear, but it
boils down to @domenic's notes in [heycam/webidl#84][2] and
[w3ctag/design-principles#93][3].

Long story short, URLs are `USVString`. This patch adjusts our
implementation to match.

[1]: https://heycam.github.io/webidl/#idl-USVString
[2]: https://github.com/heycam/webidl/issues/84#issuecomment-300857185
[3]: https://github.com/w3ctag/design-principles/issues/93#issuecomment-379816152

Change-Id: I9bf1240b421287d7d9c291b13d887ca981a66231
1 parent 93af5d9
Raw File
preloader-css-import.tentative.html 
<!DOCTYPE html>
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
<script>
    var t = async_test('Imported inline CSS is not blocked on pending CSS');
</script>
<link rel=stylesheet href="resources/dummy.css?first&pipe=trickle(d1)">
<script>
    var this_script_is_neccessary_to_block_the_inline_style_processing = true;
</script>
<style>
@import url("resources/dummy.css?second");
</style>
<script>
    window.addEventListener("load", t.step_func_done(() => {
        let entries = performance.getEntriesByType('resource');
        let first;
        let second;
        for (entry of entries) {
            if (entry.name.includes("first")) {
                first = entry;
            }
            if (entry.name.includes("second")) {
                second = entry;
            }
        }
        assert_true(first.responseEnd > second.startTime, "The second resource start time should not be blocked on the first resource response");
    }));
</script>
back to top