Make WebMIDI use blink Promise.

Source/modules/webmidi/MIDIAccess.h

Index: Source/modules/webmidi/MIDIAccess.h
diff --git a/Source/modules/webmidi/MIDIAccess.h b/Source/modules/webmidi/MIDIAccess.h
index 9f6aaaf0c853e22c63c7242311dbb0c1e48aa189..30d94f9994fa3deae0028454e5cffae0d3ee9777 100644
--- a/Source/modules/webmidi/MIDIAccess.h
+++ b/Source/modules/webmidi/MIDIAccess.h
@@ -37,6 +37,7 @@
 #include "modules/webmidi/MIDIAccessor.h"
 #include "modules/webmidi/MIDIAccessorClient.h"
 #include "modules/webmidi/MIDIInput.h"
+#include "modules/webmidi/MIDIOptions.h"
 #include "modules/webmidi/MIDIOutput.h"
 #include "wtf/RefCounted.h"
 #include "wtf/RefPtr.h"
@@ -45,13 +46,13 @@
 namespace WebCore {
 
 class ExecutionContext;
-class MIDIAccessPromise;
+class NavigatorWebMIDI;
 
 class MIDIAccess : public RefCounted<MIDIAccess>, public ScriptWrappable, public ActiveDOMObject, public EventTargetWithInlineData, public MIDIAccessorClient {
     REFCOUNTED_EVENT_TARGET(MIDIAccess);
 public:
     virtual ~MIDIAccess();
-    static PassRefPtr<MIDIAccess> create(ExecutionContext*, MIDIAccessPromise*);
+    static PassRefPtr<MIDIAccess> create(const MIDIOptions&, ExecutionContext*, NavigatorWebMIDI*);
 
     MIDIInputVector inputs() const { return m_inputs; }
     MIDIOutputVector outputs() const { return m_outputs; }
@@ -78,17 +79,19 @@ public:
     // |timeStampInMilliseconds| is in the same time coordinate system as performance.now().
     void sendMIDIData(unsigned portIndex, const unsigned char* data, size_t length, double timeStampInMilliseconds);
 
+    void startRequest();
+
 private:
-    MIDIAccess(ExecutionContext*, MIDIAccessPromise*);
+    MIDIAccess(const MIDIOptions&, ExecutionContext*, NavigatorWebMIDI*);
 
-    void startRequest();
     virtual void permissionDenied();
 
+    NavigatorWebMIDI* m_navigator;

[abarth-chromium, 2013/12/03 16:57:57]

This raw pointer is concerning.  This object looks like it can live for an
arbitrary length of time yet it has a raw pointer to another object that can
live for an arbitrary amount of time.

[yhirano, 2013/12/04 05:20:24]

We need a holder that holds pending MIDIAccessors and I think NavigatorWebMIDI
is appropriate.
I simplified the interaction between NavigatorWebMIDI and MIDIAccessor and added
comments.
m_navigator will be cleared when it goes invalid.

     MIDIInputVector m_inputs;
     MIDIOutputVector m_outputs;
-    MIDIAccessPromise* m_promise;
 
     OwnPtr<MIDIAccessor> m_accessor;
+    MIDIOptions m_options;
     bool m_hasAccess;
     bool m_sysExEnabled;
     bool m_requesting;