[debugger] Simplify deletion of DeoptimizedFrameInfo.
R=yangguo@chromium.org,jarin@chromium.org BUG= Review-Url: https://codereview.chromium.org/2112443002 Cr-Commit-Position: refs/heads/master@{#37377}
This commit is contained in:
parent
46a365faae
commit
4a8ac723ae
@ -31,15 +31,13 @@ FrameInspector::FrameInspector(JavaScriptFrame* frame,
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
FrameInspector::~FrameInspector() {
|
||||
// Get rid of the calculated deoptimized frame if any.
|
||||
if (deoptimized_frame_ != NULL) {
|
||||
Deoptimizer::DeleteDebuggerInspectableFrame(deoptimized_frame_, isolate_);
|
||||
if (deoptimized_frame_ != nullptr) {
|
||||
delete deoptimized_frame_;
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
int FrameInspector::GetParametersCount() {
|
||||
return is_optimized_ ? deoptimized_frame_->parameters_count()
|
||||
: frame_->ComputeParametersCount();
|
||||
|
@ -159,13 +159,6 @@ DeoptimizedFrameInfo* Deoptimizer::DebuggerInspectableFrame(
|
||||
return info;
|
||||
}
|
||||
|
||||
|
||||
void Deoptimizer::DeleteDebuggerInspectableFrame(DeoptimizedFrameInfo* info,
|
||||
Isolate* isolate) {
|
||||
delete info;
|
||||
}
|
||||
|
||||
|
||||
void Deoptimizer::GenerateDeoptimizationEntries(MacroAssembler* masm,
|
||||
int count,
|
||||
BailoutType type) {
|
||||
@ -173,7 +166,6 @@ void Deoptimizer::GenerateDeoptimizationEntries(MacroAssembler* masm,
|
||||
generator.Generate();
|
||||
}
|
||||
|
||||
|
||||
void Deoptimizer::VisitAllOptimizedFunctionsForContext(
|
||||
Context* context, OptimizedFunctionVisitor* visitor) {
|
||||
DisallowHeapAllocation no_allocation;
|
||||
|
@ -493,8 +493,6 @@ class Deoptimizer : public Malloced {
|
||||
static DeoptimizedFrameInfo* DebuggerInspectableFrame(JavaScriptFrame* frame,
|
||||
int jsframe_index,
|
||||
Isolate* isolate);
|
||||
static void DeleteDebuggerInspectableFrame(DeoptimizedFrameInfo* info,
|
||||
Isolate* isolate);
|
||||
|
||||
// Makes sure that there is enough room in the relocation
|
||||
// information of a code object to perform lazy deoptimization
|
||||
|
Loading…
Reference in New Issue
Block a user