Closed Bug 1158319 Opened 6 years ago Closed 6 years ago

Cache API should throw NS_ERROR_DOM_SECURITY_ERR if principal is incorrect

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: bkelly, Assigned: bkelly)

Details

Attachments

(1 file, 1 obsolete file)

Right now CacheStorage is throwing NS_ERROR_FAILURE which is gecko specific.  We can be slightly better here and throw NS_ERROR_DOM_SECURITY_ERR.  Neither is really spec'd as far as I can tell.  The SecurityError seems better for now, though.
Assignee: nobody → bkelly
Status: NEW → ASSIGNED
Attachment #8597425 - Flags: review?(ehsan)
Attachment #8597425 - Attachment is obsolete: true
Attachment #8597425 - Flags: review?(ehsan)
Try again without a bunch of unrelated changes.
Attachment #8597427 - Flags: review?(ehsan)
Raised a spec issue:

  https://github.com/slightlyoff/ServiceWorker/issues/687

We are matching what blink does for now.
Attachment #8597427 - Flags: review?(ehsan) → review+
https://hg.mozilla.org/mozilla-central/rev/3e548b8fed23
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.