Add GCController support to test_shell....

webkit/port/bindings/v8/v8_proxy.cpp

Index: webkit/port/bindings/v8/v8_proxy.cpp
===================================================================
--- webkit/port/bindings/v8/v8_proxy.cpp	(revision 5396)
+++ webkit/port/bindings/v8/v8_proxy.cpp	(working copy)
@@ -1826,8 +1826,23 @@
       V8Custom::v8DOMWindowIndexedSecurityCheck,
       v8::Integer::New(V8ClassIndex::DOMWINDOW));
 
-  // Create a new context.
-  m_context = v8::Context::New(NULL, global_template, m_global);
+  if (ScriptController::shouldExposeGCController()) {
+      v8::RegisterExtension(new v8::Extension("v8/GCController",
+          "(function v8_GCController() {"
+          "   var v8_gc;"
+          "   if (gc) v8_gc = gc;"
+          "   GCController = new Object();"
+          "   GCController.collect ="
+          "     function() {if (v8_gc) v8_gc(); };"

[Mike Belshe, 2008/11/14 01:11:54]

Cool - in JS! 

Could this be just:

if (gc)
  gc();

?

Why is the intermediate v8_gc needed?

+          " })()"));
+      const char* extension_names[] = { "v8/GCController" };
+      v8::ExtensionConfiguration extensions(1, extension_names);
+      // Create a new context.
+      m_context = v8::Context::New(&extensions, global_template, m_global);
+  } else {
+      m_context = v8::Context::New(NULL, global_template, m_global);      
+  }
+
   if (m_context.IsEmpty())
     return;