Make WebMIDI use blink Promise.

Source/modules/webmidi/NavigatorWebMIDI.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 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * 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/webmidi/NavigatorWebMIDI.h"
 
+#include "bindings/v8/DOMRequestState.h"
+#include "bindings/v8/ScriptPromise.h"
+#include "bindings/v8/ScriptPromiseResolver.h"
+#include "core/dom/DOMError.h"
 #include "core/dom/Document.h"
 #include "core/dom/ExecutionContext.h"
 #include "core/frame/Frame.h"
 #include "core/frame/Navigator.h"
-#include "modules/webmidi/MIDIAccessPromise.h"
+#include "modules/webmidi/MIDIAccess.h"
+#include "modules/webmidi/MIDIOptions.h"
 
 namespace WebCore {
 
 NavigatorWebMIDI::NavigatorWebMIDI(Frame* frame)
     : DOMWindowProperty(frame)
 {
 }
 
 NavigatorWebMIDI::~NavigatorWebMIDI()
 {
+    clearPending();
 }
 
 const char* NavigatorWebMIDI::supplementName()
 {
     return "NavigatorWebMIDI";
 }
 
 NavigatorWebMIDI* NavigatorWebMIDI::from(Navigator* navigator)
 {
     NavigatorWebMIDI* supplement = static_cast<NavigatorWebMIDI*>(Supplement<Navigator>::from(navigator, supplementName()));
     if (!supplement) {
         supplement = new NavigatorWebMIDI(navigator->frame());
         provideTo(navigator, supplementName(), adoptPtr(supplement));
     }
     return supplement;
 }
 
-PassRefPtr<MIDIAccessPromise> NavigatorWebMIDI::requestMIDIAccess(Navigator* navigator, const Dictionary& options)
+ScriptPromise NavigatorWebMIDI::requestMIDIAccess(Navigator* navigator, const Dictionary& options)
 {
     return NavigatorWebMIDI::from(navigator)->requestMIDIAccess(options);
 }
 
-PassRefPtr<MIDIAccessPromise> NavigatorWebMIDI::requestMIDIAccess(const Dictionary& options)
+ScriptPromise NavigatorWebMIDI::requestMIDIAccess(const Dictionary& options)
 {
-    if (!frame())
-        return 0;
+    if (!frame()) {

[Takashi Toyoshima, 2013/12/03 10:48:50]

optional: Can we have a layout test which confirm that requestMIDIAccess() works
also in iframe or somewhere out of main frame?

[yhirano, 2013/12/03 12:14:16]

I found that this rejection is currently not working.
Added a test and a FIXME comment.

+        ScriptPromise promise = ScriptPromise::createPending();
+        RefPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(promise);
+        resolver->reject(DOMError::create("InvalidStateError"));

[Takashi Toyoshima, 2013/12/03 10:48:50]

This is not the case described in the spec, but I think AbortError is right
error which we should return here.

[Takashi Toyoshima, 2013/12/03 11:34:49]

Sent a pull request to use AbortError here.
https://github.com/WebAudio/web-midi-api/pull/93

[yhirano, 2013/12/03 12:14:16]

Done.

+        return promise;
+    }
 
     ExecutionContext* context = frame()->document();
     ASSERT(context);
+    if (!m_requestState.get())
+        m_requestState = adoptPtr(new DOMRequestState(context));
+    ScriptPromise promise = ScriptPromise::createPending(context);
+    RefPtr<MIDIAccess> access = MIDIAccess::create(MIDIOptions(options), context, this);
+    m_pending.add(access, ScriptPromiseResolver::create(promise, context));
+    access->startRequest();
+    return promise;
+}
 
-    return MIDIAccessPromise::create(context, options);
+void NavigatorWebMIDI::didFinishRequest(MIDIAccess* access)
+{
+    // FIXME: We should care about suspend / stop.
+    RefPtr<MIDIAccess> key(access);
+    if (!m_pending.contains(key))
+        return;
+    RefPtr<ScriptPromiseResolver> resolver = m_pending.get(key);
+    m_pending.remove(key);
+
+    if (!frame()) {
+        // We can break the promise if the frame does not exist.
+        return;
+    }
+    ASSERT(m_requestState);
+    DOMRequestState::Scope scope(*m_requestState);
+    resolver->resolve(access, access->executionContext());

[Takashi Toyoshima, 2013/12/03 11:34:49]

I proposed to remove the second argument of the MIDISuccessCallback as you
implemented.
https://github.com/WebAudio/web-midi-api/issues/90

+}
+
+void NavigatorWebMIDI::didFailRequest(MIDIAccess* access, PassRefPtr<DOMError> error)
+{
+    // FIXME: We should care about suspend / stop.
+    RefPtr<MIDIAccess> key(access);
+    if (!m_pending.contains(key))
+        return;
+    RefPtr<ScriptPromiseResolver> resolver = m_pending.get(key);
+    m_pending.remove(key);
+    if (!frame()) {
+        // We can break the promise if the frame does not exist.
+        return;
+    }
+    ASSERT(m_requestState);
+    DOMRequestState::Scope scope(*m_requestState);
+    resolver->reject(error, access->executionContext());
+}
+
+void NavigatorWebMIDI::clearPending()
+{
+    // FIXME: We should care about suspend / stop.
+    HashMap<RefPtr<MIDIAccess>, RefPtr<ScriptPromiseResolver> > pending;
+    pending.swap(m_pending);
+
+    if (!frame()) {
+        // We can break promises if the frame does not exist.
+        return;
+    }
+
+    ASSERT(m_requestState);
+    for (HashMap<RefPtr<MIDIAccess>, RefPtr<ScriptPromiseResolver> >::iterator i = pending.begin();
+        i != pending.end(); ++i) {
+        DOMRequestState::Scope scope(*m_requestState);
+        MIDIAccess* access = i->key.get();
+        i->value->reject(DOMError::create("InvalidStateError"), access->executionContext());

[Takashi Toyoshima, 2013/12/03 10:48:50]

AbortError?

[yhirano, 2013/12/03 12:14:16]

Done.

+    }
 }
 
 } // namespace WebCore