1917805 - [wdspec] Add more wdspec tests for cached stylesheets (including link, import,...)

Status: RESOLVED FIXED

(Remote Protocol :: WebDriver BiDi, task, P3)

Description

[Julian Descottes [:jdescottes]]

With the initial support for cached stylesheet network events, we are probably missing events when the stylesheet is loaded via @import.
We should add a test case for this.

We should also add a test for subsequent loads of the same stylesheet in the same page (eg a page with 3 identical <link> or @import rules)

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1917805 - [wdspec] Add cached stylesheet network tests @import and duplicated stylesheets

The initial Firefox support for network events for cached stylesheets fails in 2 known cases:

• when the stylesheet is loaded via @import (no event emitted)
• when the stylesheet is duplicated in several link tags (too many events emitted)

New tests are added to cover those use cases, which should all be fixed by Bug 1916960 on Firefox side.
All the added tests already pass on Chrome canary.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/26bbaffc6022
[wdspec] Add cached stylesheet network tests @import and duplicated stylesheets r=webdriver-reviewers,arai,whimboo

Comment 3

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/26bbaffc6022

Comment 4

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/48275 for changes under testing/web-platform/tests

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR was closed without merging

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+2]]

(In reply to Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org) from comment #4)

Created web-platform-tests PR
https://github.com/web-platform-tests/wpt/pull/48275 for changes under
testing/web-platform/tests

Julian, can you please take a look at the PR? There are failures that prevent the merge.

Comment 7

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR was closed without merging

Comment 8

[Julian Descottes [:jdescottes]]

Should be fixed via https://github.com/web-platform-tests/wpt/pull/48275/commits/2569ba97c513e0116c628a4b209eb82228df9eb6

Comment 9

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR was closed without merging

Comment 10

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4854e6966768
[wpt PR 48275] - [Gecko Bug 1917805] [wdspec] Add cached stylesheet network tests @import and duplicated stylesheets, a=testonly

Comment 12

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/4854e6966768