Skip to content

Patch devtools console crash#698

Merged
petekanev merged 2 commits intomasterfrom
pete/patch-devtools-console-crash
Jan 31, 2017
Merged

Patch devtools console crash#698
petekanev merged 2 commits intomasterfrom
pete/patch-devtools-console-crash

Conversation

@petekanev
Copy link
Copy Markdown
Contributor

@petekanev petekanev commented Jan 30, 2017

Patch the inspector stack-trace implementation to not crash when an exception is thrown during script evaluation in the Chrome DevTools console.

Addresses #695

Tested in both Chrome DevTools and the official VSCode extension.

@ns-bot
Copy link
Copy Markdown

ns-bot commented Jan 30, 2017

💚

@petekanev
Copy link
Copy Markdown
Contributor Author

petekanev commented Jan 30, 2017

Alternatively this bug can be solved by explicitly defining an Isolate::Scope before calling create of the V8StackTraceImpl class to ensure a valid isolate.

@petekanev petekanev requested a review from Plamen5kov January 30, 2017 15:15
@petekanev petekanev merged commit f97a9ad into master Jan 31, 2017
@petekanev petekanev deleted the pete/patch-devtools-console-crash branch February 1, 2017 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants