Add RELEASE_ASSERT for ensuring that we have no empty context to V8WrapperInstantiationScope.

Source/bindings/v8/V8DOMWrapper.h

Index: Source/bindings/v8/V8DOMWrapper.h
diff --git a/Source/bindings/v8/V8DOMWrapper.h b/Source/bindings/v8/V8DOMWrapper.h
index a8aee01666cb3a2aaf11d2b7788d709f6c43ce74..6446dc7ebf5b175101d14e34f999a5a0c8086310 100644
--- a/Source/bindings/v8/V8DOMWrapper.h
+++ b/Source/bindings/v8/V8DOMWrapper.h
@@ -151,8 +151,7 @@ struct WrapperTypeInfo;
             // FIXME: Remove all empty creationContexts from caller sites.
             // If a creationContext is empty, we will end up creating a new object
             // in the context currently entered. This is wrong.

[haraken, 2014/06/09 06:47:03]

Would you update this FIXME to clarify why we have a RELEASE_ASSERT here?

[tasak, 2014/06/12 02:08:07]

Done.

-            if (creationContext.IsEmpty())
-                return;
+            RELEASE_ASSERT(!creationContext.IsEmpty());
             v8::Handle<v8::Context> contextForWrapper = creationContext->CreationContext();
             // For performance, we enter the context only if the currently running context
             // is different from the context that we are about to enter.