[webcrypto] Fix TODO regarding asynchronous completion of crypto operations.

Source/modules/crypto/CryptoResultImpl.cpp

 /*
  * Copyright (C) 2013 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 14 matching lines @@
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "modules/crypto/CryptoResultImpl.h"
 
 #include "bindings/v8/ScriptPromiseResolver.h"
+#include "core/dom/ExecutionContext.h"
 #include "modules/crypto/Key.h"
 #include "modules/crypto/KeyPair.h"
 #include "modules/crypto/NormalizeAlgorithm.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebArrayBuffer.h"
 #include "public/platform/WebCryptoAlgorithm.h"
 #include "wtf/ArrayBufferView.h"
 
 namespace WebCore {
 
 CryptoResultImpl::~CryptoResultImpl()
 {
     ASSERT(m_finished);
+    ASSERT(!m_promiseResolver.get());
+    ASSERT(!m_requestState.isValid());
 }
 
 PassRefPtr<CryptoResultImpl> CryptoResultImpl::create(ScriptPromise promise)
 {
-    return adoptRef(new CryptoResultImpl(promise));
+    return adoptRef(new CryptoResultImpl(activeExecutionContext(), promise));
 }
 
 void CryptoResultImpl::completeWithError()
 {
-    m_promiseResolver->reject(ScriptValue::createNull());
+    ASSERT(!m_finished);
+
+    if (canCompletePromise()) {
+        DOMRequestState::Scope scope(m_requestState);
+        m_promiseResolver->reject(ScriptValue::createNull());
+    }
+
     finish();
 }
 
 void CryptoResultImpl::completeWithBuffer(const blink::WebArrayBuffer& buffer)
 {
-    m_promiseResolver->resolve(PassRefPtr<ArrayBuffer>(buffer));
+    ASSERT(!m_finished);
+
+    if (canCompletePromise()) {
+        DOMRequestState::Scope scope(m_requestState);
+        m_promiseResolver->resolve(PassRefPtr<ArrayBuffer>(buffer));
+    }
+
     finish();
 }
 
 void CryptoResultImpl::completeWithBoolean(bool b)
 {
-    m_promiseResolver->resolve(ScriptValue::createBoolean(b));
+    ASSERT(!m_finished);
+
+    if (canCompletePromise()) {
+        DOMRequestState::Scope scope(m_requestState);
+        m_promiseResolver->resolve(ScriptValue::createBoolean(b));
+    }
+
     finish();
 }
 
 void CryptoResultImpl::completeWithKey(const blink::WebCryptoKey& key)
 {
-    m_promiseResolver->resolve(Key::create(key));
+    ASSERT(!m_finished);
+
+    if (canCompletePromise()) {
+        DOMRequestState::Scope scope(m_requestState);
+        m_promiseResolver->resolve(Key::create(key));
+    }
+
     finish();
 }
 
 void CryptoResultImpl::completeWithKeyPair(const blink::WebCryptoKey& publicKey, const blink::WebCryptoKey& privateKey)
 {
-    m_promiseResolver->resolve(KeyPair::create(publicKey, privateKey));
+    ASSERT(!m_finished);
+
+    if (canCompletePromise()) {
+        DOMRequestState::Scope scope(m_requestState);
+        m_promiseResolver->resolve(KeyPair::create(publicKey, privateKey));
+    }
+
     finish();
 }
 
-CryptoResultImpl::CryptoResultImpl(ScriptPromise promise)
-    : m_promiseResolver(ScriptPromiseResolver::create(promise))
-    , m_finished(false) { }
+CryptoResultImpl::CryptoResultImpl(ExecutionContext* context, ScriptPromise promise)
+    : ContextLifecycleObserver(context)
+    , m_promiseResolver(ScriptPromiseResolver::create(promise))
+    , m_requestState(context)
+#if !ASSERT_DISABLED
+    , m_owningThread(currentThread())
+    , m_finished(false)
+#endif
+{
+    ASSERT(toIsolate(context) == promise.isolate());
+}
 
 void CryptoResultImpl::finish()
 {
-    ASSERT(!m_finished);
+#if !ASSERT_DISABLED
     m_finished = true;
+#endif
+    clearPromiseResolver();
+}
+
+void CryptoResultImpl::clearPromiseResolver()
+{
+    m_promiseResolver.clear();
+    m_requestState.clear();
+}
+
+void CryptoResultImpl::CheckValidThread() const
+{
+    ASSERT(m_owningThread == currentThread());
+}
+
+void CryptoResultImpl::contextDestroyed()
+{
+    ContextLifecycleObserver::contextDestroyed();
+
+    // Abandon the promise without completing it when the context goes away.
+    clearPromiseResolver();
+    ASSERT(!canCompletePromise());
+}
+
+bool CryptoResultImpl::canCompletePromise() const
+{
+    CheckValidThread();
+    ExecutionContext* context = executionContext();
+    return context && !context->activeDOMObjectsAreSuspended() && !context->activeDOMObjectsAreStopped();
 }
 
 } // namespace WebCore