Make compilers agree on source position of thrown errors.

test/mjsunit/harmony/debug-step-into-constructor.js

 // Copyright 2014 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Flags: --expose-debug-as debug --harmony-classes
 
 'use strict';
 
 var Debug = debug.Debug
 var done, stepCount;
 
 function listener(event, execState, eventData, data) {
   if (event == Debug.DebugEvent.Break) {
     if (!done) {
       execState.prepareStep(Debug.StepAction.StepInto);
       var s = execState.frame().sourceLineText();
       assertTrue(s.indexOf('// ' + stepCount + '.') !== -1);
       stepCount++;
     }
   }
 };
 
 Debug.setListener(listener);
 
 
 class Base {
-  constructor() {
-    var x = 1;   // 1.
-    var y = 2;   // 2.
-    done = true; // 3.
+  constructor() {  // 1.

[Michael Starzinger, 2015/04/10 11:49:12]

Arv: Can you please advise on this change. Does this indicate a regression?

[Michael Starzinger, 2015/04/10 13:56:40]

OK, I managed to find this regression and fixed it. Please disregard my previous
comment here. Arv, I took you off the reviewer list. Sorry for the noise on this
one!

+    var x = 1;     // 2.
+    var y = 2;     // 3.
+    done = true;   // 4.
   }
 }
 
 class Derived extends Base {}
 
 
 (function TestBreakPointInConstructor() {
   done = false;
   stepCount = 1;
   var bp = Debug.setBreakPoint(Base, 0);
 
   new Base();
-  assertEquals(4, stepCount);
+  assertEquals(5, stepCount);
 
   Debug.clearBreakPoint(bp);
 })();
 
 
 (function TestDefaultConstructor() {
   done = false;
   stepCount = 1;
 
   var bp = Debug.setBreakPoint(Base, 0);
   new Derived();
-  assertEquals(4, stepCount);
+  assertEquals(5, stepCount);
 
   Debug.clearBreakPoint(bp);
 })();
 
 
 (function TestStepInto() {
   done = false;
   stepCount = 0;
 
   function f() {
     new Derived();  // 0.
   }
 
   var bp = Debug.setBreakPoint(f, 0);
   f();
-  assertEquals(4, stepCount);
+  assertEquals(5, stepCount);
 
   Debug.clearBreakPoint(bp);
 })();
 
 
 (function TestExtraIndirection() {
   done = false;
   stepCount = 0;
 
   class Derived2 extends Derived {}
 
   function f() {
     new Derived2();  // 0.
   }
 
   var bp = Debug.setBreakPoint(f, 0);
   f();
-  assertEquals(4, stepCount);
+  assertEquals(5, stepCount);
 
   Debug.clearBreakPoint(bp);
 })();
 
 
 (function TestBoundClass() {
   done = false;
   stepCount = 0;
 
   var bound = Derived.bind(null);
 
   function f() {
     new bound();  // 0.
   }
 
   var bp = Debug.setBreakPoint(f, 0);
   f();
-  assertEquals(4, stepCount);
+  assertEquals(5, stepCount);
 
   Debug.clearBreakPoint(bp);
 })();
 
 
 Debug.setListener(null);