Closed
Bug 1269042
Opened 8 years ago
Closed 8 years ago
[jsdbg2] Handle NonSyntacticVariablesObject in D.E.object
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla49
| Tracking | Status | |
|---|---|---|
| firefox49 | --- | fixed |
People
(Reporter: shu, Unassigned)
Details
Attachments
(1 file)
|
2.64 KB,
patch
|
jimb
:
review+
|
Details | Diff | Splinter Review |
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•8 years ago
|
||
Attachment #8747335 -
Flags: review?(jimb)
|
||
Comment 2•8 years 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•8 years ago
|
||
Setting checkin-needed until my SSH key is updated.
Keywords: checkin-needed
|
||
Comment 5•8 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/8793af101ab9
Status: NEW → RESOLVED
Closed: 8 years 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.
Description
•