Skip to content

Commit

Permalink
deps: fix V8 debugger bugs
Browse files Browse the repository at this point in the history
This is following-up fix for https://codereview.chromium.org/813873007/.
This bug currently breaks node-inspector client to work.

PR-URL: #494
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
  • Loading branch information
sairion authored and bnoordhuis committed Jan 18, 2015
1 parent 6f36630 commit d566ded
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions deps/v8/src/debug-debugger.js
Expand Up @@ -1887,7 +1887,7 @@ DebugCommandProcessor.prototype.resolveFrameFromScopeDescription_ =
// Get the frame for which the scope or scopes are requested.
// With no frameNumber argument use the currently selected frame.
if (scope_description && !IS_UNDEFINED(scope_description.frameNumber)) {
frame_index = scope_description.frameNumber;
var frame_index = scope_description.frameNumber;
if (frame_index < 0 || this.exec_state_.frameCount() <= frame_index) {
throw new Error('Invalid frame number');
}
Expand Down Expand Up @@ -1972,7 +1972,7 @@ DebugCommandProcessor.resolveValue_ = function(value_description) {
var value_mirror = LookupMirror(value_description.handle);
if (!value_mirror) {
throw new Error("Failed to resolve value by handle, ' #" +
mapping.handle + "# not found");
value_description.handle + "# not found");
}
return value_mirror.value();
} else if ("stringDescription" in value_description) {
Expand Down Expand Up @@ -2127,7 +2127,7 @@ DebugCommandProcessor.prototype.lookupRequest_ = function(request, response) {

// Set 'includeSource' option for script lookup.
if (!IS_UNDEFINED(request.arguments.includeSource)) {
includeSource = %ToBoolean(request.arguments.includeSource);
var includeSource = %ToBoolean(request.arguments.includeSource);
response.setOption('includeSource', includeSource);
}

Expand Down

0 comments on commit d566ded

Please sign in to comment.