Correctly compute line numbers in functions from the function constructor.

test/mjsunit/regress/regress-crbug-109362.js

Index: test/mjsunit/regress/regress-crbug-109362.js
diff --git a/test/mjsunit/regress/regress-crbug-109362.js b/test/mjsunit/regress/regress-crbug-109362.js
new file mode 100644
index 0000000000000000000000000000000000000000..38124b7981b83117eac757cf81a73e78626dd80a
--- /dev/null
+++ b/test/mjsunit/regress/regress-crbug-109362.js
@@ -0,0 +1,20 @@
+// 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.
+
+var stack;
+var f = new Function('1 + reference_error //@ sourceURL=evaltest');
+try {
+  f();
+} catch (e) {
+  stack = e.stack;
+}
+assertTrue(stack.indexOf("at eval (evaltest:1:5)") > 0);
+
+var g = new Function('\n 1 + reference_error //@ sourceURL=evaltest');

[aandrey, 2014/11/11 10:07:38]

I meant:

var g = eval('(function () {\n 1 + reference_error //@ sourceURL=evaltest\n})')

Otherwise, it is the same test case as above.

I'm asking because AFAIU this change will also affect eval(), which is wrong. I
think we should set -1 line offset only for new Function(), but not for eval().

+try {
+  g();
+} catch (e) {
+  stack = e.stack;
+}
+assertTrue(stack.indexOf("at eval (evaltest:2:6)") > 0);