Closed Bug 1220728 Opened 10 years ago Closed 10 years ago

land additional review feedback in FetchEvent error reporting from bug 1219852

Categories

(Core :: DOM: Service Workers, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla45
Tracking Status
firefox45 --- fixed

People

(Reporter: bkelly, Assigned: bkelly)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1219852 +++ This bug is to land this additional review feedback: 1) Clear pending exception after nsAutoJSString::init() failure. 2) Simply check for empty file name after DOMException::GetFilename() instead of using NS_SUCCEEDED().
Claiming r+ here since this was previously given review feedback in bug 1219852.
Attachment #8682031 - Flags: review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
removing the b2g 2.5 flag since this commit has been reverted due to an incorrect merge, sorry for the confusion
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: