Introduce KeepAliveWhilePending to ScriptPromiseResolverWithContext.

Source/modules/webmidi/MIDIAccessInitializer.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 22 matching lines @@
 
 #include "bindings/v8/ScriptFunction.h"
 #include "bindings/v8/ScriptPromise.h"
 #include "bindings/v8/ScriptPromiseResolverWithContext.h"
 #include "core/dom/DOMError.h"
 #include "core/dom/Document.h"
 #include "modules/webmidi/MIDIAccess.h"
 #include "modules/webmidi/MIDIConnectionEvent.h"
 #include "modules/webmidi/MIDIController.h"
 #include "modules/webmidi/MIDIOptions.h"
+#include "modules/webmidi/MIDIPort.h"
 
 namespace WebCore {
 
-class MIDIAccessInitializer::PostAction : public ScriptFunction {
-public:
-    static PassOwnPtr<MIDIAccessInitializer::PostAction> create(v8::Isolate* isolate, WeakPtr<MIDIAccessInitializer> owner, State state) { return adoptPtr(new PostAction(isolate, owner, state)); }
-
-private:
-    PostAction(v8::Isolate* isolate, WeakPtr<MIDIAccessInitializer> owner, State state): ScriptFunction(isolate), m_owner(owner), m_state(state) { }
-    virtual ScriptValue call(ScriptValue value) OVERRIDE
-    {
-        if (!m_owner.get())
-            return value;
-        m_owner->doPostAction(m_state);
-        return value;
-    }
-
-    WeakPtr<MIDIAccessInitializer> m_owner;
-    State m_state;
-};
+MIDIAccessInitializer::MIDIAccessInitializer(const MIDIOptions& options)
+    : m_resolver(nullptr)
+    , m_options(options)
+    , m_sysexEnabled(false)
+{
+}
 
 MIDIAccessInitializer::~MIDIAccessInitializer()
 {
-    ASSERT(m_state != Requesting);
+    cancel();
 }
 
-MIDIAccessInitializer::MIDIAccessInitializer(const MIDIOptions& options, MIDIAccess* access)
-    : m_state(Requesting)
-    , m_weakPtrFactory(this)
-    , m_options(options)
-    , m_sysexEnabled(false)
-    , m_access(access)
+void MIDIAccessInitializer::start(AsyncInitializerResolver<MIDIAccessInitializer>* resolver)
 {
+    m_resolver = resolver;
     m_accessor = MIDIAccessor::create(this);
+
+    if (!m_options.sysex) {
+        m_accessor->startSession();
+        return;
+    }
+    Document* document = toDocument(executionContext());
+    ASSERT(document);
+    MIDIController* controller = MIDIController::from(document->page());
+    if (controller) {
+        controller->requestSysexPermission(this);
+    } else {
+        m_resolver->reject(DOMError::create("SecurityError"));
+    }
 }
 
 void MIDIAccessInitializer::didAddInputPort(const String& id, const String& manufacturer, const String& name, const String& version)
 {
-    m_access->didAddInputPort(id, manufacturer, name, version);
+    m_portDescriptors.append(PortDescriptor(id, manufacturer, name, MIDIPort::MIDIPortTypeInput, version));
 }
 
 void MIDIAccessInitializer::didAddOutputPort(const String& id, const String& manufacturer, const String& name, const String& version)
 {
-    m_access->didAddOutputPort(id, manufacturer, name, version);
+    m_portDescriptors.append(PortDescriptor(id, manufacturer, name, MIDIPort::MIDIPortTypeOutput, version));
 }
 
 void MIDIAccessInitializer::didStartSession(bool success, const String& error, const String& message)
 {
-    if (success)
-        m_resolver->resolve(m_access);
-    else
+    if (success) {
+        m_resolver->resolve(MIDIAccess::create(m_accessor.release(), m_sysexEnabled, m_portDescriptors, executionContext()));
+    } else {
         m_resolver->reject(DOMError::create(error, message));
+    }
 }
 
 void MIDIAccessInitializer::setSysexEnabled(bool enable)
 {
     m_sysexEnabled = enable;
     if (enable) {
         m_accessor->startSession();
     } else {
         m_resolver->reject(DOMError::create("SecurityError"));
     }
 }
 
 SecurityOrigin* MIDIAccessInitializer::securityOrigin() const
 {
-    return m_access->executionContext()->securityOrigin();
+    return executionContext()->securityOrigin();
 }
 
 void MIDIAccessInitializer::cancel()
 {
-    if (m_state != Requesting)
-        return;
     m_accessor.clear();
-    m_weakPtrFactory.revokeAll();
     Document* document = toDocument(executionContext());
     ASSERT(document);
     MIDIController* controller = MIDIController::from(document->page());
     ASSERT(controller);
     controller->cancelSysexPermissionRequest(this);
-    m_state = Stopped;
-}
-
-ExecutionContext* MIDIAccessInitializer::executionContext() const
-{
-    return m_access->executionContext();
 }
 
 void MIDIAccessInitializer::permissionDenied()
 {
     ASSERT(isMainThread());
     m_resolver->reject(DOMError::create("SecurityError"));
 }
 
-ScriptPromise MIDIAccessInitializer::initialize(ScriptState* scriptState)
+ExecutionContext* MIDIAccessInitializer::executionContext() const
 {
-    m_resolver = ScriptPromiseResolverWithContext::create(scriptState);
-    ScriptPromise promise = m_resolver->promise();
-    promise.then(PostAction::create(scriptState->isolate(), m_weakPtrFactory.createWeakPtr(), Resolved),
-        PostAction::create(scriptState->isolate(), m_weakPtrFactory.createWeakPtr(), Stopped));
-
-    if (!m_options.sysex) {
-        m_accessor->startSession();
-        return promise;
-    }
-    Document* document = toDocument(executionContext());
-    ASSERT(document);
-    MIDIController* controller = MIDIController::from(document->page());
-    if (controller) {
-        controller->requestSysexPermission(this);
-    } else {
-        m_resolver->reject(DOMError::create("SecurityError"));
-    }
-    return promise;
-}
-
-void MIDIAccessInitializer::doPostAction(State state)
-{
-    ASSERT(m_state == Requesting);
-    ASSERT(state == Resolved || state == Stopped);
-    if (state == Resolved) {
-        m_access->initialize(m_accessor.release(), m_sysexEnabled);
-    }
-    m_accessor.clear();
-    m_weakPtrFactory.revokeAll();
-    m_state = state;
+    ASSERT(m_resolver);
+    return m_resolver->scriptState()->executionContext();
 }
 
 } // namespace WebCore