Open Bug 1391809 Opened 8 years ago Updated 1 year ago

Does JS_IsBuiltinFunctionConstructor/isBuiltinFunctionConstructor need to check for AsyncFunctionConstructor and AsyncGeneratorConstructor?

Categories

(Core :: JavaScript Engine, enhancement, P3)

enhancement

Tracking

()

Tracking Status
firefox57 --- affected

People

(Reporter: anba, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [js:tech-debt])

Looks like JSFunction::isBuiltinFunctionConstructor() needs to be updated to handle AsyncFunctionConstructor and AsyncGeneratorConstructor, right?
See bug 976151 for context when JS_IsBuiltinFunctionConstructor is used .
Priority: -- → P3
Whiteboard: [js:tech-debt]
I noticed that before but didn't fix because AsyncFunctionConstructor wasn't directly accessible as a global binding (and was out of scope to my bug at the time). I guess there are probably other ways to pull the constructor off an existing async function so we probably want to fix this.
Severity: normal → S3
Blocks: sm-runtime
Severity: S3 → N/A
You need to log in before you can comment on or make changes to this bug.