Skip to content

[wip] src, inspector: support opted-in VM contexts #14231

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
TimothyGu wants to merge 12 commits into from
Closed

[wip] src, inspector: support opted-in VM contexts #14231

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
ff88cf4
src: refactor contextify
TimothyGu Jul 14, 2017
9f12ee8
[WIP] src, inspector: support opted-in VM contexts
TimothyGu Jul 14, 2017
a08b03e
ignore already tracked contexts
TimothyGu Jul 15, 2017
2d9272c
src: add a macro for debugging inspector WS
TimothyGu Jul 17, 2017
f1ca818
test: add some tools to common
TimothyGu Jul 17, 2017
a8ebfb5
bug fixes
TimothyGu Jul 16, 2017
fe7c4a5
tests
TimothyGu Jul 16, 2017
0352a48
contextAttached
TimothyGu Jul 16, 2017
06f635b
contextAttached tests
TimothyGu Jul 16, 2017
fc7d408
auto attachment from runInContext
TimothyGu Jul 16, 2017
a0c6f68
Add docs
TimothyGu Jul 17, 2017
15a8ff8
vm: support binding a script to a context
TimothyGu Jul 17, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
bug fixes
  • Loading branch information
@TimothyGu
TimothyGu committed Jul 17, 2017
commit a8ebfb57b4fd8cb1d0d8ace17d825f97ca1156ae
3 changes: 2 additions & 1 deletion lib/inspector.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
'use strict';

const errors = require('internal/errors');
const EventEmitter = require('events');
const util = require('util');
const { isContext } = process.binding('contextify');
Expand Down Expand Up @@ -105,7 +106,7 @@ function checkSandbox(sandbox) {

let ctxIdx = 1;
function attachContext(sandbox, {
name = `vm Module Context ${ctxIdx}`,
name = `vm Module Context ${ctxIdx++}`,
origin
} = {}) {
checkSandbox(sandbox);
Expand Down
2 changes: 1 addition & 1 deletion lib/internal/errors.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,7 @@ E('ERR_NAPI_CONS_PROTOTYPE_OBJECT', 'Constructor.prototype must be an object');
E('ERR_NO_CRYPTO', 'Node.js is not compiled with OpenSSL crypto support');
E('ERR_PARSE_HISTORY_DATA', 'Could not parse history data in %s');
E('ERR_SANDBOX_NOT_CONTEXTIFIED',
'Provided sandbox must have been converted to a context')
'Provided sandbox must have been converted to a context');
E('ERR_SOCKET_ALREADY_BOUND', 'Socket is already bound');
E('ERR_SOCKET_BAD_TYPE',
'Bad socket type specified. Valid types are: udp4, udp6');
Expand Down