Closed Bug 1251311 Opened 5 years ago Closed 5 years ago

Stop pretending that JS::DescribeScriptedCaller can throw an exception in worker code.

Categories

(Core :: DOM: Workers, defect)

Product:
Core
Component:
DOM: Workers
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla47
Tracking Flags:
Tracking Status
firefox47 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

Details

(Whiteboard: btpp-active)

Attachments

(1 file, 1 obsolete file)

JS::DescribeScriptedCaller can't throw JS exceptions. Adjust some callers accordingly
2.04 KB, patch
khuey
: review+
Details | Diff | Splinter Review 
It can't.  False just means "there is no scripted caller".
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #8723654 - Attachment is obsolete: true
Attachment #8723654 - Flags: review?(khuey)
Whiteboard: btpp-active
https://hg.mozilla.org/mozilla-central/rev/d6c6898a18f6
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox47: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
You need to log in before you can comment on or make changes to this bug.