Debugger: require debugger to be active when dealing with breaks.

test/mjsunit/debug-stepin-positions.js

 // Copyright 2008 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 15 matching lines @@
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 // Flags: --expose-debug-as debug --nocrankshaft
 // Get the Debug object exposed from the debug context global object.
 Debug = debug.Debug
 
 function DebuggerStatement() {
   debugger;  /*pause*/
 }
 
-function TestCase(fun, frame_number) {
+function TestCase(fun, frame_number, line_number) {
   var exception = false;
   var codeSnippet = undefined;
   var resultPositions = undefined;
   var step = 0;
 
   function listener(event, exec_state, event_data, data) {
     try {
       if (event == Debug.DebugEvent.Break ||
           event == Debug.DebugEvent.Exception) {
         if (step++ > 0) return;
@@ skipping 10 matching lines @@
     function assertHasLineMark(mark, frame) {
         var line = frame.sourceLineText();
         if (!mark.exec(frame.sourceLineText())) {
             throw new Error("Line " + line + " should contain mark " + mark);
         }
     }
   }
 
   Debug.setListener(listener);
 
+  var breakpointId;
+  if (line_number) breakpointId = Debug.setBreakPoint(fun, line_number);
+
   fun();
 
+  if (line_number) Debug.clearBreakPoint(breakpointId);
+
   Debug.setListener(null);
 
   assertTrue(!exception, exception);
 
   var expectedPositions = {};
   var markPattern = new RegExp("/\\*#\\*/", "g");
 
   var matchResult;
   while ( (matchResult = markPattern.exec(codeSnippet)) ) {
     expectedPositions[matchResult.index] = true;
@@ skipping 30 matching lines @@
   }
   assertFalse(unexpectedPositionFound, "Found unexpected position: " +
       decoratedResult);
 }
 
 function TestCaseWithDebugger(fun) {
   TestCase(fun, 1);
 }
 
 function TestCaseWithBreakpoint(fun, line_number, frame_number) {
-  var breakpointId = Debug.setBreakPoint(fun, line_number);
-  TestCase(fun, frame_number);
-  Debug.clearBreakPoint(breakpointId);
+  TestCase(fun, frame_number, line_number);
 }
 
 function TestCaseWithException(fun, frame_number) {
   Debug.setBreakOnException();
   TestCase(fun, frame_number);
   Debug.clearBreakOnException();
 }
 
 
 // Test cases.
@@ skipping 83 matching lines @@
 })(Object);
 TestCaseWithDebugger(fun);
 
 // Global function, global object construction, calls before pause point.
 var fun = (function(p) {
   return function() {
     var res = [ Math.abs(new Object()), DebuggerStatement(), Math./*#*/abs(4), /*#*/new Object()./*#*/toString() ]; /*positions*/
   };
 })(Object);
 TestCaseWithDebugger(fun);