Bug 1452483 - Set correct line number for JS evaluation. r?maja_zf
Marionette incorrectly sets the JavaScript context line number to 0.
The line number is provided to us in in the input, so we should
use this. The default fallback if line is not provided is 0 as before.
MozReview-Commit-ID: 8gOt9r4awee
--- a/testing/marionette/evaluate.js
+++ b/testing/marionette/evaluate.js
@@ -155,17 +155,17 @@ evaluate.sandbox = function(sb, script,
}
// timeout and unload handlers
scriptTimeoutID = setTimeout(timeoutHandler, timeout);
sb.window.onunload = unloadHandler;
let res;
try {
- res = Cu.evalInSandbox(src, sb, "1.8", file, 0);
+ res = Cu.evalInSandbox(src, sb, "1.8", file, line);
} catch (e) {
let err = new JavaScriptError(e, {
fnName: "execute_script",
file,
line,
script,
});
reject(err);