From da43c458f6fc7c960b8ff7e691d7400c83f481f9 Mon Sep 17 00:00:00 2001 From: Shu-yu Guo Date: Tue, 12 Nov 2013 10:51:09 -0800 Subject: [PATCH] Bug 936143 - Part 1: Add Debugger.Script.lineCount test where the script's source notes end in newline notes. (r=jimb) --- .../jit-test/tests/debug/Script-startLine.js | 20 ++++++++++++++++++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/js/src/jit-test/tests/debug/Script-startLine.js b/js/src/jit-test/tests/debug/Script-startLine.js index 9801230f6e9b..4d3c49950102 100644 --- a/js/src/jit-test/tests/debug/Script-startLine.js +++ b/js/src/jit-test/tests/debug/Script-startLine.js @@ -8,9 +8,11 @@ dbg.onDebuggerStatement = function (frame) { assertEq(typeof frame.script.url, 'string'); }; -function test(f) { +function test(f, manualCount) { start = count = g.first = g.last = undefined; f(); + if (manualCount) + g.last = g.first + manualCount - 1; assertEq(start, g.first); assertEq(count, g.last + 1 - g.first); print(start, count); @@ -41,3 +43,19 @@ g.eval("function f2() {\n" + "}\n"); test(g.f2); test(g.f2); + +// Having a last = Error().lineNumber forces a setline srcnote, so test a +// function that ends with newline srcnotes. +g.eval("/* Any copyright is dedicated to the Public Domain.\n" + + " http://creativecommons.org/publicdomain/zero/1.0/ */\n" + + "\n" + + "function secondCall() { first = Error().lineNumber;\n" + + " debugger;\n" + + " // Comment\n" + + " eval(\"42;\");\n" + + " function foo() {}\n" + + " if (true) {\n" + + " foo();\n" + // <- this is +6 and must be within the extent + " }\n" + + "}"); +test(g.secondCall, 7);