Closed Bug 792358 Opened 12 years ago Closed 12 years ago

Pass JSScript to Detecting to avoid an expensive currentScript call

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 786801

People

(Reporter: jandem, Assigned: jandem)

References

Details

Attachments

(1 file)

Attached patch PatchSplinter Review
Looking up the current script is slow with IonMonkey. Since all callers of Detecting already do this, it's straight-forward and much faster to just pass the script to Detecting. (Waldo, sorry for the small conflict with bug 792108..)
Attachment #662479 - Flags: review?(jwalden+bmo)
Blocks: 792354
Incidentally, bug 786801 does, I believe, the same thing. It is r+'d and hopefully about to land.
(In reply to Luke Wagner [:luke] from comment #1) > Incidentally, bug 786801 does, I believe, the same thing. It is r+'d and > hopefully about to land. OK thanks, closing this one as duplicate then :)
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
Attachment #662479 - Flags: review?(jwalden+bmo)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: