Fixing wrong extension ID in Activity Log entries.

Source/bindings/v8/V8WindowShell.cpp

Index: Source/bindings/v8/V8WindowShell.cpp
diff --git a/Source/bindings/v8/V8WindowShell.cpp b/Source/bindings/v8/V8WindowShell.cpp
index 9209dba2114ec1c60c1452f0136bfcf123d8f440..6aa0dd4bfcfcfa26d4678329791bcc2a18928c7a 100644
--- a/Source/bindings/v8/V8WindowShell.cpp
+++ b/Source/bindings/v8/V8WindowShell.cpp
@@ -216,7 +216,8 @@ bool V8WindowShell::initialize()
             setInjectedScriptContextDebugId(context, m_frame->script().contextDebugId(mainWindow->context()));
     }
 
-    m_scriptState->world().setActivityLogger(V8DOMActivityLogger::activityLogger(m_world->worldId()));
+    m_scriptState->setActivityLogger(V8DOMActivityLogger::activityLogger(m_world->worldId(),
+        m_frame->document() ? m_frame->document()->baseURI() : KURL()));

[haraken, 2014/05/08 02:38:32]

Is it possible that m_frame->document() is 0? If it's possible, what's an
expected behavior?

[pmarch, 2014/05/08 17:55:48]

I do not know if m_frame->document() can be 0 here. I could not force
m_frame->document() to be 0 when tested this patch. I placed this check just in
case as I saw that the code below does check for m_frame->document() == 0.
However, the code below calls updateDocument() which may set m_frame->document()
to 0, if this is the case my check is redundant.

On 2014/05/08 02:38:32, haraken wrote:

     if (!installDOMWindow()) {
         disposeContext(DoNotDetachGlobal);
         return false;