Closed Bug 1158265 Opened 9 years ago Closed 9 years ago

service-workers/cache-storage wpt tests expect to add/addAll to throw NetworkError for invalid scheme

Categories

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

Product:
Testing
Component:
web-platform-tests
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: bkelly, Assigned: bkelly)

References

(
URL
)

Details

There are tests in the service-workers/cache-storage tests that expect add()/addAll() to throw NetworErrorif an invalid scheme is used.  This is incorrect.  The spec requires TypeError.

For example:

  "If r's url's scheme is not one of "http" and "https", or r's method is not `GET`, return a promise rejected with a TypeError."

In step 3.2 here:

  https://slightlyoff.github.io/ServiceWorker/spec/service_worker/index.html#cache-addAll
Assignee: nobody → bkelly
URL: https://critic.hoppipolla.co.uk/r/4835
Status: NEW → ASSIGNED
Merged from upstream in bug 1161535.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Depends on: 1161535
No longer depends on: 1158262
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.