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

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

 /*
  * Copyright (C) 2008, 2009, 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 273 matching lines @@
   // If conditional features for window have been queued before the V8 context
   // was ready, then inject them into the context now
   if (m_world->isMainWorld()) {
     installPendingConditionalFeaturesOnWindow(m_scriptState.get());
   }
 
   return true;
 }
 
 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
   // setupWindowPrototypeChain for the structure of the prototype chain of
   // the global object.
   v8::Local<v8::ObjectTemplate> globalTemplate =
       V8Window::domTemplate(m_isolate, *m_world)->InstanceTemplate();
   CHECK(!globalTemplate.IsEmpty());
 
   // FIXME: It's not clear what the right thing to do for remote frames is.
@@ skipping 267 matching lines @@
       .ToChecked();
 }
 
 void WindowProxy::updateSecurityOrigin(SecurityOrigin* origin) {
   if (!isContextInitialized())
     return;
   setSecurityToken(origin);
 }
 
 }  // namespace blink