45b4522e40
This is in preparation to the removal of the FullCodeGenerator, we no longer need the ability to stress the underlying implementation. R=rmcilroy@chromium.org BUG=v8:6409 Cq-Include-Trybots: master.tryserver.v8:v8_linux_noi18n_rel_ng Change-Id: Iad3177d6de4a68b57c12a770b6e85ed7a9710254 Reviewed-on: https://chromium-review.googlesource.com/584747 Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> Commit-Queue: Michael Starzinger <mstarzinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#47276}
68 lines
1.8 KiB
JavaScript
68 lines
1.8 KiB
JavaScript
// Copyright 2017 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.
|
|
|
|
// Test that declaring local variables in IIFEs works with
|
|
// side-effect free debug-evaluate.
|
|
|
|
Debug = debug.Debug
|
|
|
|
var exception = null;
|
|
|
|
function listener(event, exec_state, event_data, data) {
|
|
if (event != Debug.DebugEvent.Break) return;
|
|
try {
|
|
function success(expectation, source) {
|
|
assertEquals(expectation,
|
|
exec_state.frame(0).evaluate(source, true).value());
|
|
}
|
|
function fail(source) {
|
|
assertThrows(() => exec_state.frame(0).evaluate(source, true),
|
|
EvalError);
|
|
}
|
|
// Declaring 'a' sets a property to the global object.
|
|
fail("var a = 3");
|
|
exec_state.frame(0).evaluate("var a = 2", false);
|
|
assertEquals(2, a);
|
|
// Wrapping into an IIFE would be fine, since 'a' is local.
|
|
success(100,
|
|
`(function(x) {
|
|
var a = 0;
|
|
for (var i = 0; i < x; i++) {
|
|
a += x;
|
|
}
|
|
return a;
|
|
})(10);`);
|
|
success(100,
|
|
`(x => {
|
|
let a = 0;
|
|
for (var i = 0; i < x; i++) {
|
|
a += x;
|
|
}
|
|
return a;
|
|
})(10);`);
|
|
// Not using 'var' to declare would make the access go to global object.
|
|
fail( `(function(x) {
|
|
a = 0;
|
|
for (var i = 0; i < x; i++) {
|
|
a += x;
|
|
}
|
|
return a;
|
|
})(10);`);
|
|
} catch (e) {
|
|
exception = e;
|
|
print(e, e.stack);
|
|
};
|
|
};
|
|
|
|
// Add the debug event listener.
|
|
Debug.setListener(listener);
|
|
|
|
function f() {
|
|
debugger;
|
|
};
|
|
|
|
f();
|
|
|
|
assertNull(exception);
|