1186833 - "File not found" issue when running "register-same-scope-different-script-url.https.html" test

Status: RESOLVED FIXED

(Testing :: web-platform-tests, defect)

Description

[Noemí Freire (:noemi)]

Test run such as |./mach web-platform-tests _mozilla/service-workers/service-worker/register-same-scope-different-script-url.https.html|

Actual Result:
"File not found"
Firefox can't find the file at https://web-platform.test:8443/_mozilla/service-workers/service-worker/resources/scope/register-new-script-controller.

Comment 1

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Attachment: MozReview Request: Bug 1186833 - register-same-scope-different-script-url.https.html fixes. r?bkelly

Bug 1186833 - register-same-scope-different-script-url.https.html fixes. r?bkelly

Add empty html file to allow onload to be fired until Bug 1201632 is fixed.
Update web-platform-tests expected data

Comment 2

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Comment on attachment 8658376 [details]
MozReview Request: Bug 1186833 - register-same-scope-different-script-url.https.html fixes. r?bkelly

Bug 1186833 - register-same-scope-different-script-url.https.html fixes. r?bkelly

Add empty html file to allow onload to be fired until Bug 1201632 is fixed.
Update web-platform-tests expected data

Comment 3

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Updated to remove the empty html file since Bug 1201632 is going to be fixed. I'll update the commit message, please review this.

Comment 4

[Noemí Freire (:noemi)]

Patch provided by Nikhil so assigning this to him. Please feel free to change it as needed. Thanks!

Comment 5

[Ben Kelly [:bkelly, not reviewing]]

Comment on attachment 8658376 [details]
MozReview Request: Bug 1186833 - register-same-scope-different-script-url.https.html fixes. r?bkelly

https://reviewboard.mozilla.org/r/18589/#review16717

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/6f01338ecce9

Comment 7

[Nigel Babu [:nigelb]]

According to bugzilla, this is directly dependent on bug 1201632. The patch in that bug caused bustage, so I had to back this one out as well so as to not cause more bustage because of dependencies.

Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/63f9f2c90830.

Comment 8

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=8478afbeae96

Comment 9

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/5b5a5360cc2924cbdf0e16c610d61efc764361f1
Bug 1186833 - register-same-scope-different-script-url.https.html fixes. r=bkelly

Comment 10

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/5b5a5360cc29

Comment 11

[Noemí Freire (:noemi)]

Hi,

just checked in m-c (e7d613b3bcfe revision) and the tests successfully run. Thanks for fixing it!

Summary

Harness status: OK

Found 5 tests
5 Pass
Details
Result	Test Name
Pass	Register different scripts concurrently	
Pass	Register then register new script URL	
Pass	Register then register new script URL that 404s	
Pass	Register then register new script that does not install	
Pass	Register same-scope new script url effect on controller