[DevTools] Removed ScriptState from createScriptCallStackForConsole

Source/bindings/core/v8/ScriptCallStackFactory.cpp

Index: Source/bindings/core/v8/ScriptCallStackFactory.cpp
diff --git a/Source/bindings/core/v8/ScriptCallStackFactory.cpp b/Source/bindings/core/v8/ScriptCallStackFactory.cpp
index 00aca0e79d76d11ff5d4ca33205ba22d56df8f27..d176bcefc28d6fab9f182941b9df1f8aa3ffb05c 100644
--- a/Source/bindings/core/v8/ScriptCallStackFactory.cpp
+++ b/Source/bindings/core/v8/ScriptCallStackFactory.cpp
@@ -111,11 +111,12 @@ PassRefPtrWillBeRawPtr<ScriptCallStack> createScriptCallStack(size_t maxStackSiz
     return createScriptCallStack(stackTrace, maxStackSize, emptyStackIsAllowed, isolate);
 }
 
-PassRefPtrWillBeRawPtr<ScriptCallStack> createScriptCallStackForConsole(ScriptState* scriptState, size_t maxStackSize)
+PassRefPtrWillBeRawPtr<ScriptCallStack> createScriptCallStackForConsole(size_t maxStackSize)
 {
     size_t stackSize = 1;
     if (InspectorInstrumentation::hasFrontends()) {
-        if (InspectorInstrumentation::consoleAgentEnabled(scriptState->executionContext()))
+        v8::Isolate* isolate = v8::Isolate::GetCurrent();
+        if (InspectorInstrumentation::consoleAgentEnabled(currentExecutionContext(isolate)))

[vsevik, 2014/08/11 15:52:03]

These lines might be needed here, please check with yurys@
if (!isolate->InContext())
    return nullptr;
v8::HandleScope scope(isolate);

[kozyatinskiy1, 2014/08/11 16:05:34]

First check added. @yurus, could you please answer: do we really need a scope in
this case?

[aandrey, 2014/08/11 17:38:03]

1) Calling currentExecutionContext() when not in context should be fine since
https://codereview.chromium.org/16943006

2) You don't use v8::Handle<>'s, so handle scope not needed.

[kozyatinskiy1, 2014/08/11 17:50:42]

Thank you. Furthermore, this test may be useful here for a quick exit from the
function.

             stackSize = maxStackSize;
     }
     return createScriptCallStack(stackSize);