Handle rejected promises better for workers.

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 89 matching lines @@
     // and its fields will be traced when scanning the stack.
     WorkerScriptController* m_controller;
     WorkerGlobalScopeExecutionState* m_outerState;
 };
 
 WorkerScriptController::WorkerScriptController(WorkerGlobalScope& workerGlobalScope)
     : m_isolate(0)
     , m_workerGlobalScope(workerGlobalScope)
     , m_executionForbidden(false)
     , m_executionScheduledToTerminate(false)
+    , m_rejectedPromises(RejectedPromises::create())
     , m_globalScopeExecutionState(0)
 {
     m_isolate = V8PerIsolateData::initialize();
     V8Initializer::initializeWorker(m_isolate);
     m_world = DOMWrapperWorld::create(m_isolate, WorkerWorldId);
     m_interruptor = adoptPtr(new V8IsolateInterruptor(m_isolate));
     ThreadState::current()->addInterruptor(m_interruptor.get());
     ThreadState::current()->registerTraceDOMWrappers(m_isolate, V8GCController::traceDOMWrappers);
 }
 
@@ skipping 13 matching lines @@
     }
 
 private:
     explicit IsolateCleanupTask(v8::Isolate* isolate) : m_isolate(isolate)  { }
 
     v8::Isolate* m_isolate;
 };
 
 WorkerScriptController::~WorkerScriptController()
 {
+    m_rejectedPromises->dispose();

[haraken, 2015/02/08 01:53:55]

Won't ~WorkerScriptController() be called in some destructor of an on-heap
object?

I guess it would be safe to call dispose() in WorkerGlobalScope::dispose().

[sof, 2015/02/08 06:57:39]

It is called from there; the OwnPtr<WorkerScriptController> is cleared via
clearScript().

+    m_rejectedPromises.clear();
+
     ThreadState::current()->removeInterruptor(m_interruptor.get());
 
     m_world->dispose();
 
     // The corresponding call to didStartRunLoop() is in WorkerThread::initialize().
     // See http://webkit.org/b/83104#c14 for why this is here.
     m_workerGlobalScope.thread()->didStopRunLoop();
 
     if (isContextInitialized())
         m_scriptState->disposePerContextData();
@@ skipping 154 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(m_isolate, errorMessage));
 }
 
 } // namespace blink