Improve Dart Debugger performance and robustness by creating Dart wrappers using the standard SetNa…

Source/bindings/v8/V8Binding.cpp

Index: Source/bindings/v8/V8Binding.cpp
diff --git a/Source/bindings/v8/V8Binding.cpp b/Source/bindings/v8/V8Binding.cpp
index 93f7167a1fcdd3619a403b4c7d001d730da3efa0..fff76b945dd1343e25aedae47087a11eb7fc7452 100644
--- a/Source/bindings/v8/V8Binding.cpp
+++ b/Source/bindings/v8/V8Binding.cpp
@@ -311,6 +311,10 @@ DOMWindow* toDOMWindow(v8::Handle<v8::Context> context)
     if (!window.IsEmpty())
         return V8DOMWindow::toNative(window);
     window = global->FindInstanceInPrototypeChain(V8DOMWindow::GetTemplate(context->GetIsolate(), IsolatedWorld));
+    if (window.IsEmpty()) {

[vsm, 2013/07/15 18:40:29]

Is this something to upstream sooner rather than later?  I'm worried about
losing these changes on Blink rolls.

[Jacob, 2013/07/15 22:23:54]

The inspector tests will fail without this change as console.log won't work so
we won't lose it with Blink rolls.
That said, I don't think this change is right.  Hopefully Anton can point me to
a better long term solution to get the V8 isolate we are talking to in the
correct state.

+        // TODO(jacobr): find a better solution.
+        return toDOMWindow(v8::Context::GetEntered());
+    }
     ASSERT(!window.IsEmpty());
     return V8DOMWindow::toNative(window);
 }