Simon Zünd
39852e0b06
This is a reland of commit c7cbb4139b
Relanding as is. Was reverted due to a test failure that turned out
to be a flake that still reproduced after the revert landed.
Flako run: https://ci.chromium.org/ui/p/v8/builders/try.triggered/v8_flako/b8758034788224082417/overview
Original change's description:
> [debug] Remove 'value unavailble' experiment
>
> The experiment was enabled in July 2023 and we haven't heard any
> negative feedback so we can safely remove the experiment now.
>
> R=bmeurer@chromium.org
>
> Bug: chromium:1328681
> Change-Id: I6613a4e8e741d8c16881ec1adeb01e039244fac7
> Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/5224284
> Commit-Queue: Simon Zünd <szuend@chromium.org>
> Reviewed-by: Benedikt Meurer <bmeurer@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#91968}
Bug: chromium:1328681
Change-Id: I651e2a20c6612fc138246aa7e61daab285b57795
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/5233640
Commit-Queue: Simon Zünd <szuend@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Auto-Submit: Simon Zünd <szuend@chromium.org>
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#91978}
55 lines
1.8 KiB
JavaScript
55 lines
1.8 KiB
JavaScript
// Copyright 2015 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: --noanalyze-environment-liveness
|
|
|
|
|
|
Debug = debug.Debug
|
|
|
|
listenerComplete = false;
|
|
breakPointCount = 0;
|
|
exception = null;
|
|
|
|
function listener(event, exec_state, event_data, data) {
|
|
if (event != Debug.DebugEvent.Break) return;
|
|
try {
|
|
breakPointCount++;
|
|
if (breakPointCount == 1) {
|
|
// Break point in initializer for parameter `a`, invoked by
|
|
// initializer for parameter `b`
|
|
assertEquals('default', exec_state.frame(0).evaluate('mode').value());
|
|
assertThrows(() => exec_state.frame(1).evaluate('b'));
|
|
assertThrows(() => exec_state.frame(1).evaluate('c'));
|
|
} else if (breakPointCount == 2) {
|
|
// Break point in IIFE initializer for parameter `c`
|
|
assertEquals('modeFn', exec_state.frame(1).evaluate('a.name').value());
|
|
assertEquals('default', exec_state.frame(1).evaluate('b').value());
|
|
assertThrows(() => exec_state.frame(1).evaluate('c'));
|
|
} else if (breakPointCount == 3) {
|
|
// Break point in function body --- `c` parameter is shadowed
|
|
assertEquals('modeFn', exec_state.frame(0).evaluate('a.name').value());
|
|
assertEquals('default', exec_state.frame(0).evaluate('b').value());
|
|
assertEquals('local', exec_state.frame(0).evaluate('d').value());
|
|
}
|
|
} catch (e) {
|
|
exception = e;
|
|
}
|
|
};
|
|
|
|
// Add the debug event listener.
|
|
Debug.setListener(listener);
|
|
|
|
function f(a = function modeFn(mode) { debugger; return mode; },
|
|
b = a("default"),
|
|
c = (function() { debugger; })()) {
|
|
var d = 'local';
|
|
debugger;
|
|
};
|
|
|
|
f();
|
|
|
|
// Make sure that the debug event listener vas invoked.
|
|
assertEquals(3, breakPointCount);
|
|
assertNull(exception);
|