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
ignore already tracked contexts
  • Loading branch information
@TimothyGu
TimothyGu committed Jul 15, 2017
commit a08b03eebe7bcbefbee1655229730db7ad0c7483
17 changes: 14 additions & 3 deletions src/inspector_agent.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -418,6 +418,7 @@ void AttachContext(const v8::FunctionCallbackInfo<v8::Value>& args) {
auto info = new node::inspector::ContextInfo(
contextify_context->context(), group_id, name, origin,
"{\"isDefault\":false}");
// Ignore error.
env->inspector_agent()->ContextCreated(info);
}

Expand Down Expand Up @@ -620,9 +621,19 @@ Agent::Agent(Environment* env) : parent_env_(env),
Agent::~Agent() {
}

void Agent::ContextCreated(const node::inspector::ContextInfo* info) {
bool Agent::ContextCreated(const node::inspector::ContextInfo* info) {
auto isolate = parent_env_->isolate();
auto it = std::find_if(
contexts_.begin(), contexts_.end(),
[&] (const node::inspector::ContextInfo*& cur) {
return cur->context(isolate) == info->context(isolate);
});
if (it != contexts_.end()) {
return false;
}
contexts_.push_back(info);
client_->contextCreated(info);
return true;
}

void Agent::ContextDestroyed(Local<Context> context) {
Expand All @@ -646,10 +657,10 @@ bool Agent::Start(v8::Platform* platform, const char* path,
client_ =
std::unique_ptr<NodeInspectorClient>(
new NodeInspectorClient(parent_env_, platform));
ContextCreated(
CHECK(ContextCreated(
new node::inspector::ContextInfo(
parent_env_->context(), CONTEXT_GROUP_ID, "Node.js Main Context",
nullptr, "{\"isDefault\":true}"));
nullptr, "{\"isDefault\":true}")));
platform_ = platform;
CHECK_EQ(0, uv_async_init(uv_default_loop(),
&start_io_thread_async,
Expand Down
2 changes: 1 addition & 1 deletion src/inspector_agent.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ class Agent {
// Stop and destroy io_
void Stop();

void ContextCreated(const node::inspector::ContextInfo* info);
bool ContextCreated(const node::inspector::ContextInfo* info);
void ContextDestroyed(v8::Local<v8::Context> context);

bool IsStarted() { return !!client_; }
Expand Down