Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2)

Unified Diff: src/debug.cc

Issue 297373002: Fix memory leak in the debugger. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | src/isolate.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/debug.cc
diff --git a/src/debug.cc b/src/debug.cc
index 900821beaf0471fee3f0f5bb68ba9dfcb866e4ea..0af63f61fdcd63ea3731250c6726d6bf9da46fed 100644
--- a/src/debug.cc
+++ b/src/debug.cc
@@ -805,6 +805,8 @@ bool Debug::Load() {
void Debug::Unload() {
+ ClearAllBreakPoints();
+
// Return debugger is not loaded.
if (!IsLoaded()) return;
@@ -3150,7 +3152,6 @@ void Debugger::UpdateState() {
activate = debug->Load();
} else if (debug->IsLoaded() && !activate) {
isolate_->compilation_cache()->Enable();
- debug->ClearAllBreakPoints();
debug->Unload();
}
is_active_ = activate;
« no previous file with comments | « no previous file | src/isolate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698