1638309 - Better document the debugger-eval scope hack

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P3)

Description

[Ted Campbell [:tcampbell]]

The clever hack here should be better explained. There are more details in https://phabricator.services.mozilla.com/D75455#2296583 and we should clean them up and put into tree.

Comment 1

[Ted Campbell [:tcampbell]]

(Before I lose this off clipboard..)

Eval Scope    Eval Env         Frame Env    Frame Scope
============  =============    =========    =============

NonSyntactic
   |
   v
  null        DebugEnvProxy                 LexicalScope
                    |                            |
                    v                            v
              DebugEnvProxy --> CallObj --> FunctionScope
                    |              |             |
                    v              v             v
                   ...            ...           ...

Comment 2

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1638309 - Better document the debugger-eval scope hack r?tcampbell

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a87db3b698e1
Better document the debugger-eval scope hack r=tcampbell DONTBUILD

Comment 4

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/a87db3b698e1