If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

[jsdbg2] Handle NonSyntacticVariablesObject in D.E.object

RESOLVED FIXED in Firefox 49

Status

()

Core
JavaScript Engine
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: shu, Unassigned)

Tracking

unspecified
mozilla49
Points:
---

Firefox Tracking Flags

(firefox49 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

a year ago
mrbkap hit this while using the browser debugger to debug frame scripts. NonSyntacticVariablesObjects are stand-ins for the global for frame scripts, because creating new globals is too expensive. It's not a declarative scope and D.E.object lacked a case to deal with it.
(Reporter)

Comment 1

a year ago
Created attachment 8747335 [details] [diff] [review]
Handle NonSyntacticVariablesObject in Debugger.Environment.object.
Attachment #8747335 - Flags: review?(jimb)

Comment 2

a year ago
Comment on attachment 8747335 [details] [diff] [review]
Handle NonSyntacticVariablesObject in Debugger.Environment.object.

Review of attachment 8747335 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks very much! I'm glad this was able to be tested in the shell.
Attachment #8747335 - Flags: review?(jimb) → review+
(Reporter)

Comment 3

a year ago
Setting checkin-needed until my SSH key is updated.
Keywords: checkin-needed

Comment 4

a year ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/8793af101ab9
Keywords: checkin-needed

Comment 5

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/8793af101ab9
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.