Drop V8RecursionScope dependency on ExecutionContext

Source/bindings/core/v8/WorkerScriptController.cpp

Index: Source/bindings/core/v8/WorkerScriptController.cpp
diff --git a/Source/bindings/core/v8/WorkerScriptController.cpp b/Source/bindings/core/v8/WorkerScriptController.cpp
index 8ebafdfe1fcc773312a944b369fbc9062c7cc352..c82bcd45152f446f33e1aa047335e94b42fd30af 100644
--- a/Source/bindings/core/v8/WorkerScriptController.cpp
+++ b/Source/bindings/core/v8/WorkerScriptController.cpp
@@ -130,7 +130,7 @@ public:
 
     virtual void postCleanup()
     {
-        V8PerIsolateData::dispose(m_isolate);
+        V8PerIsolateData::destroy(m_isolate);
     }
 
 private:
@@ -150,6 +150,8 @@ WorkerScriptController::~WorkerScriptController()
     // See http://webkit.org/b/83104#c14 for why this is here.
     blink::Platform::current()->didStopWorkerRunLoop(blink::WebWorkerRunLoop(m_workerGlobalScope.thread()));
 
+    V8PerIsolateData::willBeDestroyed(m_isolate);

[haraken, 2014/09/11 23:21:18]

Nit: I'd put this after line 156. Conceptually PerIsolateData's cleanup should
be done at the very end of other cleanups (i.e., after disposePerContextData).

[jsbell, 2014/09/12 23:27:48]

Done.

+
     if (isContextInitialized())
         m_scriptState->disposePerContextData();