Fix memory leak in the debugger.

src/debug.cc

 // Copyright 2012 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.
 
 #include "v8.h"
 
 #include "api.h"
 #include "arguments.h"
 #include "bootstrapper.h"
 #include "code-stubs.h"
@@ skipping 787 matching lines @@
   // Check for caught exceptions.
   if (caught_exception) return false;
 
   debug_context_ = Handle<Context>::cast(
       isolate_->global_handles()->Create(*context));
   return true;
 }
 
 
 void Debug::Unload() {
+  ClearAllBreakPoints();
+
   // Return debugger is not loaded.
   if (!IsLoaded()) return;
 
   // Clear the script cache.
   DestroyScriptCache();
 
   // Clear debugger context global handle.
   GlobalHandles::Destroy(Handle<Object>::cast(debug_context_).location());
   debug_context_ = Handle<Context>();
 }
@@ skipping 2325 matching lines @@
   bool activate = message_handler_ != NULL ||
                   !event_listener_.is_null() ||
                   debug->InDebugger();
   if (!is_active_ && activate) {
     // Note that the debug context could have already been loaded to
     // bootstrap test cases.
     isolate_->compilation_cache()->Disable();
     activate = debug->Load();
   } else if (debug->IsLoaded() && !activate) {
     isolate_->compilation_cache()->Enable();
-    debug->ClearAllBreakPoints();
     debug->Unload();
   }
   is_active_ = activate;
   // At this point the debug context is loaded iff the debugger is active.
   ASSERT(debug->IsLoaded() == is_active_);
 }
 
 
 // Calls the registered debug message handler. This callback is part of the
 // public API.
@@ skipping 375 matching lines @@
   logger_->DebugEvent("Put", message.text());
 }
 
 
 void LockingCommandMessageQueue::Clear() {
   LockGuard<Mutex> lock_guard(&mutex_);
   queue_.Clear();
 }
 
 } }  // namespace v8::internal