Trace JSOP_INCNAME/JSOP_NAMEINC for closures

RESOLVED FIXED

Status

()

Core
JavaScript Engine
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: dmandelin, Assigned: dmandelin)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

9 years ago
See bug 492918.
(Assignee)

Comment 1

9 years ago
Created attachment 391256 [details] [diff] [review]
Patch

I think it's done, but I want to let the try server go over it before asking for review.
(Assignee)

Comment 2

9 years ago
Created attachment 391676 [details] [diff] [review]
Patch 2
Attachment #391256 - Attachment is obsolete: true
Attachment #391676 - Flags: review?(gal)

Comment 3

9 years ago
Comment on attachment 391676 [details] [diff] [review]
Patch 2

|tracked| looks weird. I think we usually use '"tracked"' but I am not sure. I like NameResult. This is going to conflict the hell out of jorendorff's patch in my queue, so adding him.
Attachment #391676 - Flags: review?(gal) → review+
(Assignee)

Comment 4

9 years ago
Pushed to TM as e3d2c39d5aa4.

I left |tracked| in because we have stuff like |this| in there already. But if you really don't like it, I won't mind at all if you change it.
Whiteboard: fixed-in-tracemonkey

Comment 5

9 years ago
http://hg.mozilla.org/mozilla-central/rev/e3d2c39d5aa4
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.