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

BUG=369451

Review URL: https://codereview.chromium.org/315043007

git-svn-id: svn://svn.chromium.org/blink/trunk@176002 bbb929c8-8fbe-4397-9dbb-9b2b20218538

third_party/WebKit/Source/bindings/v8/CustomElementWrapper.cpp

@@ -90,10 +90,8 @@ v8::Handle<v8::Object> CustomElementWrapper<ElementType, WrapperType>::wrap(Pass
 {
     ASSERT(DOMDataStore::getWrapper<V8Element>(element.get(), isolate).IsEmpty());
 
-    // FIXME: creationContext.IsEmpty() should never happen. Remove
-    // this when callers (like InspectorController::inspect) are fixed
-    // to never pass an empty creation context.
-    v8::Handle<v8::Context> context = creationContext.IsEmpty() ? isolate->GetCurrentContext() : creationContext->CreationContext();
+    ASSERT(!creationContext.IsEmpty());
+    v8::Handle<v8::Context> context = creationContext->CreationContext();
 
     if (!element->isUpgradedCustomElement() || DOMWrapperWorld::world(context).isIsolatedWorld())
         return createUpgradeCandidateWrapper(element.get(), creationContext, isolate, createSpecificWrapper);

third_party/WebKit/Source/bindings/v8/V8DOMWrapper.h

@@ -148,11 +148,10 @@ struct WrapperTypeInfo;
             : m_didEnterContext(false)
             , m_context(isolate->GetCurrentContext())
         {
-            // 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.
-            if (creationContext.IsEmpty())
-                return;
+            // creationContext should not be empty. Because if we have an
+            // empty creationContext, we will end up creating
+            // a new object in the context currently entered. This is wrong.
+            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.