15cb3fde7d
Reason for revert: Didn't help... Original issue's description: > Revert of [debugger] do not restart frames that reference new.target for liveedit. (patchset #1 id:1 of https://codereview.chromium.org/1493363002/ ) > > Reason for revert: > [Sheriff] Speculative revert for https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/3225 > > Original issue's description: > > [debugger] do not restart frames that reference new.target for liveedit. > > > > R=mstarzinger@chromium.org > > > > Committed: https://crrev.com/6fca870240bdbb07a365189b5eb0c98fa65b3682 > > Cr-Commit-Position: refs/heads/master@{#32572} > > TBR=mstarzinger@chromium.org,yangguo@chromium.org > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > > Committed: https://crrev.com/1a61dab34b9849f3f70a42ce69317e22758c53a1 > Cr-Commit-Position: refs/heads/master@{#32582} TBR=mstarzinger@chromium.org,yangguo@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1492393003 Cr-Commit-Position: refs/heads/master@{#32587}
74 lines
1.8 KiB
JavaScript
74 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: --expose-debug-as debug --allow-natives-syntax
|
|
|
|
// Test that live-editing a frame above one that uses new.target succeeds.
|
|
|
|
Debug = debug.Debug
|
|
var wrapper_calls = 0;
|
|
var construct_calls = 0;
|
|
var exceptions = 0;
|
|
var results = [];
|
|
var replace_again;
|
|
|
|
eval(`
|
|
function LogNewTarget(arg) {
|
|
construct_calls++;
|
|
results.push(new.target);
|
|
}
|
|
function Wrapper() {
|
|
wrapper_calls++;
|
|
ReplaceOnce();
|
|
new LogNewTarget(true);
|
|
}
|
|
`);
|
|
|
|
function Replace(fun, original, patch) {
|
|
%ExecuteInDebugContext(function() {
|
|
var change_log = [];
|
|
try {
|
|
var script = Debug.findScript(fun);
|
|
var patch_pos = script.source.indexOf(original);
|
|
Debug.LiveEdit.TestApi.ApplySingleChunkPatch(
|
|
script, patch_pos, original.length, patch, change_log);
|
|
} catch (e) {
|
|
exceptions++;
|
|
}
|
|
});
|
|
}
|
|
|
|
function ReplaceOnce(x) {
|
|
if (replace_again) {
|
|
replace_again = false;
|
|
Replace(Wrapper, "true", "false");
|
|
}
|
|
}
|
|
|
|
function Revert() {
|
|
Replace(Wrapper, "false", "true");
|
|
}
|
|
|
|
replace_again = true;
|
|
ReplaceOnce();
|
|
Wrapper();
|
|
Revert();
|
|
assertEquals(1, construct_calls);
|
|
assertEquals(1, wrapper_calls);
|
|
assertEquals(0, exceptions); // Replace succeeds
|
|
assertEquals([LogNewTarget], results);
|
|
|
|
Wrapper();
|
|
assertEquals(2, construct_calls);
|
|
assertEquals(2, wrapper_calls);
|
|
assertEquals(0, exceptions); // Replace succeeds
|
|
assertEquals([LogNewTarget, LogNewTarget], results);
|
|
|
|
replace_again = true;
|
|
Wrapper();
|
|
assertEquals(3, construct_calls);
|
|
assertEquals(4, wrapper_calls); // Restarts
|
|
assertEquals(0, exceptions); // Replace succeeds
|
|
assertEquals([LogNewTarget, LogNewTarget, LogNewTarget], results);
|