Remove onactivate-script-error.https.html.ini file - follow-up to bug 1189678

RESOLVED INVALID

Status

defect
RESOLVED INVALID
4 years ago
4 years ago

People

(Reporter: noemi, Assigned: nsm)

Tracking

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox43 affected)

Details

Attachments

(1 attachment)

Reporter

Description

4 years ago
Follow-up to bug 1189678 to remove the testing/web-platform/mozilla/meta/service-workers/service-worker/onactivate-script-error.https.html.ini file (please refer to https://bugzilla.mozilla.org/show_bug.cgi?id=1189678#c10)
Bug 1198636 - Remove incorrect ini file. r?bkelly

Bug 1189678 forgot to remove the ini file itself though the expectation data was updated.
Attachment #8658446 - Flags: review?(bkelly)
Reporter

Comment 2

4 years ago
Patch provided by Nikhil so assigning this to him. Please feel free to change it as needed. Thanks!
Assignee: nobody → nsm.nikhil
Status: NEW → ASSIGNED
Comment on attachment 8658446 [details]
MozReview Request: Bug 1198636 - Remove incorrect ini file. r?bkelly

https://reviewboard.mozilla.org/r/18607/#review16719
Attachment #8658446 - Flags: review?(bkelly) → review+
Actually, this file should not be removed. The test tests that activation succeeds even if the worker throws an error. Various worker scripts throw errors, which are reported by the browser leading to the test harness ending in an error (the last error raised by the last worker). So we should expect the harness itself to fail.
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.