Reviewers: Yang, eseidel,

Description:
Fix Unhandled ReferenceError in debug-debugger.js

This fixes following exception in Sky on attempt to set a breakpoint
"Unhandled: Uncaught ReferenceError: break_point is not defined"
I think this happens in Sky but not in Chrome because Sky scripts are executed
in strict mode.

BUG=None
LOG=N

Please review this at https://codereview.chromium.org/741683002/

Base URL: https://chromium.googlesource.com/v8/v8.git@master

Affected files (+1, -1 lines):
  M src/debug-debugger.js


Index: src/debug-debugger.js
diff --git a/src/debug-debugger.js b/src/debug-debugger.js
index fc4c5da5b7a21e2619e95c3e12f81a41c4dd4806..9be2b044d31647b30f04c164d7f83c487e9a3145 100644
--- a/src/debug-debugger.js
+++ b/src/debug-debugger.js
@@ -432,7 +432,7 @@ ScriptBreakPoint.prototype.set = function (script) {
   if (IS_NULL(position)) return;

   // Create a break point object and set the break point.
-  break_point = MakeBreakPoint(position, this);
+  var break_point = MakeBreakPoint(position, this);
   break_point.setIgnoreCount(this.ignoreCount());
   var actual_position = %SetScriptBreakPoint(script, position,
                                              this.position_alignment_,


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to