Make WebMIDI use blink Promise.

Source/bindings/v8/MIDIAccessResolver.h

Index: Source/bindings/v8/MIDIAccessResolver.h
diff --git a/LICENSE b/Source/bindings/v8/MIDIAccessResolver.h
similarity index 63%
copy from LICENSE
copy to Source/bindings/v8/MIDIAccessResolver.h
index 70bcb8ad118978579fa055f7ecc99604930900ce..9cc62b2fda0cf5bac69922776d9aa0a641128a4f 100644
--- a/LICENSE
+++ b/Source/bindings/v8/MIDIAccessResolver.h
@@ -28,3 +28,44 @@
 // 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.

[abarth-chromium, 2014/03/20 00:43:46]

Please use new-style copyright blocks:

http://dev.chromium.org/blink/coding-style#TOC-License

[yhirano, 2014/03/20 02:40:51]

Done.

+
+#ifndef MIDIAccessResolver_h
+#define MIDIAccessResolver_h
+
+#include "bindings/v8/ScriptPromiseResolver.h"
+#include "wtf/OwnPtr.h"
+#include "wtf/RefCounted.h"
+
+namespace v8 {
+class isolate;

[abarth-chromium, 2014/03/20 00:43:46]

This must not be needed because the class is named Isolate...

[yhirano, 2014/03/20 02:40:51]

Done.

+} // namespace v8
+
+namespace WebCore {
+
+class DOMError;
+class DOMWrapperWorld;
+class ExecutionContext;
+class MIDIAccess;
+
+class MIDIAccessResolver {
+    WTF_MAKE_NONCOPYABLE(MIDIAccessResolver);
+public:
+    static PassOwnPtr<MIDIAccessResolver> create(PassRefPtr<ScriptPromiseResolver> resolver, v8::Isolate* isolate)
+    {
+        return adoptPtr(new MIDIAccessResolver(resolver, isolate));
+    }
+    ~MIDIAccessResolver();
+
+    void resolve(MIDIAccess*, ExecutionContext*);
+    void reject(DOMError*, ExecutionContext*);
+
+private:
+    MIDIAccessResolver(PassRefPtr<ScriptPromiseResolver>, v8::Isolate*);
+
+    RefPtr<ScriptPromiseResolver> m_resolver;
+    RefPtr<DOMWrapperWorld> m_world;

[abarth-chromium, 2014/03/20 00:43:46]

Is RefPtr<DOMWrapperWorld> allowed anymore?  haraken would know.  You should
double-check with him before landing.

[yhirano, 2014/03/20 02:40:51]

Currently FrameLoader and ErrorEvent use RefPtr<DOMWrapperWorld> in
/Source/core. 

haraken said "we should use RefPtr<DOMWrapper> when we store DOMWrapperWorld in
heap allocated objects..." in his CL[1] landed this month.

[1]: https://src.chromium.org/viewvc/blink?revision=168874&view=revision

[haraken, 2014/03/20 04:37:16]

RefPtr<DOMWrapperWorld> is ok here. Dan and I discussed whether this should be a
RefPtr or a raw pointer and concluded that a RefPtr is slightly better
(https://codereview.chromium.org/186633002/).

In the future, I'll implement a better mechanism to wrap DOMWrapperWorld and
ExecutionContext.

+};
+
+} // namespace WebCore
+
+#endif // #ifndef MIDIAccessResolver_h