binding: Removes dead code in WindowProxy::createContext().

third_party/WebKit/Source/bindings/core/v8/WindowProxy.cpp

Index: third_party/WebKit/Source/bindings/core/v8/WindowProxy.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/WindowProxy.cpp b/third_party/WebKit/Source/bindings/core/v8/WindowProxy.cpp
index ea3eb2be02594c9960ff10f840ae5001e5ed2b73..fa2a6818da952bcdd5a0aa1b3167688590a552e1 100644
--- a/third_party/WebKit/Source/bindings/core/v8/WindowProxy.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/WindowProxy.cpp
@@ -291,21 +291,6 @@ bool WindowProxy::initialize() {
 }
 
 void WindowProxy::createContext() {
-  // FIXME: This should be a null check of m_frame->client(), but there are
-  // still some edge cases
-  // that this fails to catch during frame detach.
-  if (m_frame->isLocalFrame() &&
-      !toLocalFrame(m_frame)->loader().documentLoader()) {
-    // TODO(yukishiino): Remove this if-clause entirely once we are sure that
-    // it's safe.  There seems no case that we hit this if-clause.  Plus,
-    // createContext() is responsible just to create a new v8::Context, and it's
-    // technically possible to create it without a document loader.
-    // If we really want to limit the context creation under some condition,
-    // it must come with the right reason described in a comment.
-    NOTREACHED();
-    return;
-  }
-
   // Create a new v8::Context with the window object as the global object
   // (aka the inner global).  Reuse the global proxy object (aka the outer
   // global) if it already exists.  See the comments in