[v8-dev] Fix leak of a global handle in Debug::Load(). (issue6048003)

[vegorov]

Reviewers: Søren Gjesse,

Description:
Fix leak of a global handle in Debug::Load().
Bootstrapper::CreateEnvironment() returns a global handle to the created  
context
so there is no need to create another one explicitly.

BUG=v8:1006

Please review this at http://codereview.chromium.org/6048003/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M src/debug.cc


Index: src/debug.cc
diff --git a/src/debug.cc b/src/debug.cc
index  
7709eb0e93de98bdd60e9de9b3868dd4136ff4ef..ca3c1db790b0ead96669fd49793dafdfef70dcaf  
100644
--- a/src/debug.cc
+++ b/src/debug.cc
@@ -858,7 +858,7 @@ bool Debug::Load() {
   if (caught_exception) return false;

   // Debugger loaded.
-  debug_context_ = Handle<Context>::cast(GlobalHandles::Create(*context));
+  debug_context_ = context;

   return true;
 }


--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev