wpt icon indicating copy to clipboard operation
wpt copied to clipboard

Add a few tests for whether CSS subresources are critical

Open gsnedders opened this issue 8 years ago • 8 comments

tests for https://github.com/w3c/csswg-drafts/issues/1088

these should be moved somewhere better once we know what directory to put them in

gsnedders avatar Apr 10 '17 14:04 gsnedders

View the complete job log.

Firefox (nightly channel)

Testing web-platform-tests at revision 5048fcbb7d4f38e5699c85af809fe35483db5cbb Using browser at version BuildID 20170409194145; SourceStamp 45692c884fdd5136a64fb2f8a61a0c8183b69331 Starting 10 test iterations All results were stable

All results

5 tests ran
/unspecified-css-subresources/background-image.html
Subtest Results Messages
OK
load waits for background-image PASS
/unspecified-css-subresources/font-face-unused.html
Subtest Results Messages
OK
load doesn't wait for unused @font-face PASS
/unspecified-css-subresources/font-face-used.html
Subtest Results Messages
OK
load waits for used @font-face PASS
/unspecified-css-subresources/import.html
Subtest Results Messages
OK
load waits for @import PASS
/unspecified-css-subresources/list-style-image.html
Subtest Results Messages
OK
load waits for list-style PASS

w3c-bots avatar Apr 10 '17 15:04 w3c-bots

View the complete job log.

Chrome (unstable channel)

Testing web-platform-tests at revision 5048fcbb7d4f38e5699c85af809fe35483db5cbb Using browser at version 59.0.3063.4 dev Starting 10 test iterations All results were stable

All results

5 tests ran
/unspecified-css-subresources/background-image.html
Subtest Results Messages
OK
load waits for background-image PASS
/unspecified-css-subresources/font-face-unused.html
Subtest Results Messages
OK
load doesn't wait for unused @font-face PASS
/unspecified-css-subresources/font-face-used.html
Subtest Results Messages
OK
load waits for used @font-face PASS
/unspecified-css-subresources/import.html
Subtest Results Messages
OK
load waits for @import PASS
/unspecified-css-subresources/list-style-image.html
Subtest Results Messages
OK
load waits for list-style PASS

w3c-bots avatar Apr 10 '17 15:04 w3c-bots

w3c-test:mirror

gsnedders avatar Apr 10 '17 15:04 gsnedders

w3c-test:mirror

gsnedders avatar Apr 10 '17 15:04 gsnedders

These tests are now available on w3c-test.org

w3c-bots avatar Apr 10 '17 15:04 w3c-bots

There are no owners for this pull request. Please reach out on W3C's irc server (irc.w3.org, port 6665) on channel #testing (web client) to get help with this. Thank you!

wpt-pr-bot avatar Jun 12 '18 17:06 wpt-pr-bot

w3c-test:mirror

gsnedders avatar Jun 12 '18 17:06 gsnedders

While the CSS WG has still not resolved the above issue, these tests do at this point show identical behaviour across all three major browser engines, hence on the whole I'd be in favour of landing these as tentative tests if someone wants to review them.

gsnedders avatar Jul 23 '22 18:07 gsnedders

For some reason this thread was buried in my inbox and it doesn't look there's been much activity in a while. Are you still seeking review @gsnedders? Has there been any more consensus or spec updates here?

domfarolino avatar Nov 25 '22 18:11 domfarolino

Has there been any more consensus or spec updates here?

Not that I'm aware of; that said, I'm still in favour of review (and landing) the tests which pass in all three major browser engines, as we may as well have tentative tests that assert the current intersection behaviour (and ensure we don't move away from interoperability there).

gsnedders avatar Nov 27 '22 02:11 gsnedders

Another variation for these tests that could be useful to include in this PR is: moving all of the <style> block contents to a <link rel=stylesheet> and seeing if the link element's load event is indeed blocked when the Document's load event is blocked.

In https://github.com/w3c/csswg-drafts/issues/1088#issuecomment-427126353, Hiroshige points out that apparently Chrome blocks the Document's load event in some cases where the <link> element's load event is not blocked, which is pretty interesting. It would be good to know what browsers do here more generally.

domfarolino avatar Nov 28 '22 03:11 domfarolino

Another variation for these tests that could be useful to include in this PR is: moving all of the <style> block contents to a <link rel=stylesheet> and seeing if the link element's load event is indeed blocked when the Document's load event is blocked.

I'll fix the nits you mentioned above, but I probably won't expand the coverage of this (I don't really immediately have the time to actually work on this right now).

gsnedders avatar Nov 28 '22 16:11 gsnedders

Let me know whenever you do and I'm happy to re-review

domfarolino avatar Dec 10 '22 03:12 domfarolino