f2fd4923f3
At the moment when the static private method is unused in source code (either explicitly or through eval) but is accessed at runtime through the debugger, and there are no other potential references to the class variable in the source code otherwise, the reference to the class variable is lost here since the class variable would not be context-allocated, then we could not rebuild a proper brand check for it. For now, a ReferenceError would be thrown and the method is considered "optimized away", similar to how unused ordinary methods in closures work. Before this patch it would DCHECK when generating bytecode for the debugger instead of throwing errors. Bug: v8:9839, v8:8330 Change-Id: I5d63131a7bdba141d01a3e6459bc27d0f5953c1a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2095637 Commit-Queue: Joyee Cheung <joyee@igalia.com> Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> Reviewed-by: Shu-yu Guo <syg@chromium.org> Cr-Commit-Position: refs/heads/master@{#66734}
80 lines
2.5 KiB
JavaScript
80 lines
2.5 KiB
JavaScript
// Copyright 2020 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: --harmony-private-methods
|
|
|
|
let { session, contextGroup, Protocol } = InspectorTest.start(
|
|
"Test accessing unused private methods at runtime"
|
|
);
|
|
|
|
const script = `
|
|
function run() {
|
|
class A {
|
|
#instanceMethod() { return 2; }
|
|
static #staticMethod() { return 1; }
|
|
static testStatic() { debugger; }
|
|
testInstance() { debugger; }
|
|
};
|
|
A.testStatic();
|
|
const a = new A;
|
|
a.testInstance();
|
|
}`;
|
|
|
|
contextGroup.addScript(script);
|
|
|
|
InspectorTest.runAsyncTestSuite([
|
|
async function testScopesPaused() {
|
|
Protocol.Debugger.enable();
|
|
|
|
// Do not await here, instead oncePaused should be awaited.
|
|
Protocol.Runtime.evaluate({ expression: 'run()' });
|
|
|
|
InspectorTest.log('Get privateProperties of A in testStatic()');
|
|
let {
|
|
params: { callFrames }
|
|
} = await Protocol.Debugger.oncePaused(); // inside A.testStatic()
|
|
let frame = callFrames[0];
|
|
let { result } = await Protocol.Runtime.getProperties({
|
|
objectId: frame.this.objectId
|
|
});
|
|
InspectorTest.logObject(result.privateProperties);
|
|
|
|
// Variables not referenced in the source code are currently
|
|
// considered "optimized away".
|
|
InspectorTest.log('Access A.#staticMethod() in testStatic()');
|
|
({ result } = await Protocol.Debugger.evaluateOnCallFrame({
|
|
expression: 'A.#staticMethod();',
|
|
callFrameId: callFrames[0].callFrameId
|
|
}));
|
|
InspectorTest.logObject(result);
|
|
|
|
InspectorTest.log('Access this.#staticMethod() in testStatic()');
|
|
({ result } = await Protocol.Debugger.evaluateOnCallFrame({
|
|
expression: 'this.#staticMethod();',
|
|
callFrameId: callFrames[0].callFrameId
|
|
}));
|
|
InspectorTest.logObject(result);
|
|
|
|
Protocol.Debugger.resume();
|
|
({ params: { callFrames } } = await Protocol.Debugger.oncePaused()); // a.testInstatnce();
|
|
frame = callFrames[0];
|
|
|
|
InspectorTest.log('get privateProperties of a in testInstance()');
|
|
({ result } = await Protocol.Runtime.getProperties({
|
|
objectId: frame.this.objectId
|
|
}));
|
|
InspectorTest.logObject(result.privateProperties);
|
|
|
|
InspectorTest.log('Evaluating this.#instanceMethod() in testInstance()');
|
|
({ result } = await Protocol.Debugger.evaluateOnCallFrame({
|
|
expression: 'this.#instanceMethod();',
|
|
callFrameId: callFrames[0].callFrameId
|
|
}));
|
|
InspectorTest.logObject(result);
|
|
|
|
Protocol.Debugger.resume();
|
|
Protocol.Debugger.disable();
|
|
}
|
|
]);
|