Closed
Bug 1198636
Opened 10 years ago
Closed 10 years ago
Remove onactivate-script-error.https.html.ini file - follow-up to bug 1189678
Categories
(Testing :: web-platform-tests, defect)
Testing
web-platform-tests
Tracking
(firefox43 affected)
RESOLVED
INVALID
Tracking | Status | |
---|---|---|
firefox43 | --- | affected |
People
(Reporter: noemi, Assigned: nsm)
References
Details
Attachments
(1 file)
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)
Assignee | ||
Comment 1•10 years ago
|
||
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•10 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 3•10 years ago
|
||
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+
Assignee | ||
Comment 6•10 years ago
|
||
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
Closed: 10 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•