Fix Runtime.executionContextCreated for crafted iframes.

Source/core/inspector/PageRuntimeAgent.cpp

 /*
  * Copyright (C) 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 87 matching lines @@
     m_mainWorldContextCreated = true;
 
     if (!m_enabled)
         return;
     ASSERT(m_frontend);
 
     if (frame == m_inspectedPage->mainFrame()) {
         m_scriptStateToId.clear();
         m_frontend->executionContextsCleared();
     }
+    if (!frame->script().initializeMainWorld()) {

[aandrey, 2014/10/29 15:18:50]

I don't think we should initialize main world from inspector code.
Maybe this is a bad naming and should be actually named
isMainWorldContextInitialized() ?

[eustas, 2014/10/30 10:08:04]

Actually, we need to initialize main world sooner or later, to provide ability
to manipulate the page.
This patch makes is explicit. Implicitly main world is initialized with
ScriptState::forMainWorld

+        // Reusing existing context.
+        String frameId = m_pageAgent->frameId(frame);
+        addExecutionContextToFrontend(ScriptState::forMainWorld(frame), true, "", frameId);
+    }
+}
+
+void PageRuntimeAgent::didCreateMainWorldContext(LocalFrame* frame, ScriptState* scriptState, SecurityOrigin* origin)
+{
+    if (!m_enabled)
+        return;
+    ASSERT(m_frontend);
     String frameId = m_pageAgent->frameId(frame);
-    addExecutionContextToFrontend(ScriptState::forMainWorld(frame), true, "", frameId);
+    addExecutionContextToFrontend(scriptState, true, origin->toRawString(), frameId);
 }
 
 void PageRuntimeAgent::didCreateIsolatedContext(LocalFrame* frame, ScriptState* scriptState, SecurityOrigin* origin)
 {
     if (!m_enabled)
         return;
     ASSERT(m_frontend);
     String frameId = m_pageAgent->frameId(frame);
     addExecutionContextToFrontend(scriptState, false, origin->toRawString(), frameId);
 }
@@ skipping 54 matching lines @@
             scriptStatesToRemove.append(scriptState);
             m_frontend->executionContextDestroyed(it->value);
         }
     }
     m_scriptStateToId.removeAll(scriptStatesToRemove);
     injectedScriptManager()->discardInjectedScriptsFor(window);
 }
 
 } // namespace blink