Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

/service-workers/service-worker/svg-target-reftest.https.html times out #11353

Open
zcorpan opened this issue Jun 5, 2018 · 6 comments
Open
Assignees

Comments

@zcorpan
Copy link
Member

zcorpan commented Jun 5, 2018

From #11269

service-workers/service-worker/svg-target-reftest.https.html

@upsuper why does this test time out? Running it locally it seems the reftest-wait class is removed.

@upsuper
Copy link
Member

upsuper commented Jun 5, 2018

It seems if it times out, there is probably some issue around either the service worker registration or page loading? Maybe somehow the given scope has other service workers registered which leads to some conflicts?

I have no idea... Maybe @wanderview has some idea since he wrote this test.

@wanderview
Copy link
Member

How are you running this test? Firefox passes the test in mozilla's infrastructure and when run from our ./mach web-platform-tests CLI tool.

@wanderview
Copy link
Member

wanderview commented Jun 5, 2018

I probably will need @jgraham's help here since I don't know how our mozilla setup differs from other configs.

@jgraham
Copy link
Contributor

jgraham commented Jun 5, 2018

The test passes for me locally using wpt run firefox service-workers/service-worker/svg-target-reftest.https.html, with either --reftest-internal or --reftest-external.

@jugglinmike
Copy link
Contributor

This test was timing out across four browsers when we ran it against revision 6736e3f46b of WPT back in mid-April. In the latest test collection, this test only timed out in Safari:

https://wpt.fyi/results/service-workers/service-worker/svg-target-reftest.https.html?sha=2d0250896e

It may be an intermittent timeout, or maybe some change to the implementations or the results collector corrected the problem.

@zcorpan
Copy link
Member Author

zcorpan commented Jun 5, 2018

OK. @cdumez do you want to take a look for Safari? Locally for me it removes the reftest-wait class in Safari 11.1 and TP.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants