Closed Bug 1315940 Opened 8 years ago Closed 8 years ago

Simplify JSFunction::existingScriptForInlinedFunction

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: jandem, Assigned: jandem)

References

Details

In bug 976536 I added a workaround for an issue we found in the wild, but I'm pretty sure that got fixed so we can simplify this function a bit.

Bug 1312491 also touches this code so I'll wait for that to land first.
Using lazy->maybeScript() is more code but it triggers the read barrier on LazyScript::script_. That's no big deal but the current code avoids that and works fine and isn't that unreasonable so whatever.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.