Drop V8RecursionScope dependency on ExecutionContext

Source/bindings/core/v8/WorkerScriptController.cpp

 /*
  * Copyright (C) 2009, 2012 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 112 matching lines @@
 // are destroyed.
 class IsolateCleanupTask : public ThreadState::CleanupTask {
 public:
     static PassOwnPtr<IsolateCleanupTask> create(v8::Isolate* isolate)
     {
         return adoptPtr(new IsolateCleanupTask(isolate));
     }
 
     virtual void postCleanup()
     {
-        V8PerIsolateData::dispose(m_isolate);
+        V8PerIsolateData::destroy(m_isolate);
     }
 
 private:
     explicit IsolateCleanupTask(v8::Isolate* isolate) : m_isolate(isolate)  { }
 
     v8::Isolate* m_isolate;
 };
 
 WorkerScriptController::~WorkerScriptController()
 {
     ThreadState::current()->removeInterruptor(m_interruptor.get());
 
     m_world->dispose();
 
     // The corresponding call to didStartWorkerRunLoop is in
     // WorkerThread::initialize().
     // 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();
 
     ThreadState::current()->addCleanupTask(IsolateCleanupTask::create(m_isolate));
 }
 
 bool WorkerScriptController::initializeContextIfNeeded()
 {
     v8::HandleScope handleScope(m_isolate);
 
@@ skipping 146 matching lines @@
 
 void WorkerScriptController::rethrowExceptionFromImportedScript(PassRefPtrWillBeRawPtr<ErrorEvent> errorEvent, ExceptionState& exceptionState)
 {
     const String& errorMessage = errorEvent->message();
     if (m_globalScopeExecutionState)
         m_globalScopeExecutionState->m_errorEventFromImportedScript = errorEvent;
     exceptionState.rethrowV8Exception(V8ThrowException::createGeneralError(errorMessage, m_isolate));
 }
 
 } // namespace blink