Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(160)

Side by Side Diff: Source/bindings/v8/MIDIAccessResolver.h

Issue 77773003: Make WebMIDI use blink Promise. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // 2 //
3 // The Chromium Authors can be found at 3 // The Chromium Authors can be found at
4 // http://src.chromium.org/svn/trunk/src/AUTHORS 4 // http://src.chromium.org/svn/trunk/src/AUTHORS
5 // 5 //
6 // Redistribution and use in source and binary forms, with or without 6 // Redistribution and use in source and binary forms, with or without
7 // modification, are permitted provided that the following conditions are 7 // modification, are permitted provided that the following conditions are
8 // met: 8 // met:
9 // 9 //
10 // * Redistributions of source code must retain the above copyright 10 // * Redistributions of source code must retain the above copyright
11 // notice, this list of conditions and the following disclaimer. 11 // notice, this list of conditions and the following disclaimer.
12 // * Redistributions in binary form must reproduce the above 12 // * Redistributions in binary form must reproduce the above
13 // copyright notice, this list of conditions and the following disclaimer 13 // copyright notice, this list of conditions and the following disclaimer
14 // in the documentation and/or other materials provided with the 14 // in the documentation and/or other materials provided with the
15 // distribution. 15 // distribution.
16 // * Neither the name of Google Inc. nor the names of its 16 // * Neither the name of Google Inc. nor the names of its
17 // contributors may be used to endorse or promote products derived from 17 // contributors may be used to endorse or promote products derived from
18 // this software without specific prior written permission. 18 // this software without specific prior written permission.
19 // 19 //
20 // THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS 20 // THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
21 // "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT 21 // "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
22 // LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR 22 // LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
23 // A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT 23 // A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
24 // OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, 24 // OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
25 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT 25 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
26 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, 26 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
27 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY 27 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
28 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT 28 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
29 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE 29 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
30 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 30 // 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
yhirano 2014/03/20 02:40:51 Done.
31
32 #ifndef MIDIAccessResolver_h
33 #define MIDIAccessResolver_h
34
35 #include "bindings/v8/ScriptPromiseResolver.h"
36 #include "wtf/OwnPtr.h"
37 #include "wtf/RefCounted.h"
38
39 namespace v8 {
40 class isolate;
abarth-chromium 2014/03/20 00:43:46 This must not be needed because the class is named
yhirano 2014/03/20 02:40:51 Done.
41 } // namespace v8
42
43 namespace WebCore {
44
45 class DOMError;
46 class DOMWrapperWorld;
47 class ExecutionContext;
48 class MIDIAccess;
49
50 class MIDIAccessResolver {
51 WTF_MAKE_NONCOPYABLE(MIDIAccessResolver);
52 public:
53 static PassOwnPtr<MIDIAccessResolver> create(PassRefPtr<ScriptPromiseResolve r> resolver, v8::Isolate* isolate)
54 {
55 return adoptPtr(new MIDIAccessResolver(resolver, isolate));
56 }
57 ~MIDIAccessResolver();
58
59 void resolve(MIDIAccess*, ExecutionContext*);
60 void reject(DOMError*, ExecutionContext*);
61
62 private:
63 MIDIAccessResolver(PassRefPtr<ScriptPromiseResolver>, v8::Isolate*);
64
65 RefPtr<ScriptPromiseResolver> m_resolver;
66 RefPtr<DOMWrapperWorld> m_world;
abarth-chromium 2014/03/20 00:43:46 Is RefPtr<DOMWrapperWorld> allowed anymore? harak
yhirano 2014/03/20 02:40:51 Currently FrameLoader and ErrorEvent use RefPtr<DO
haraken 2014/03/20 04:37:16 RefPtr<DOMWrapperWorld> is ok here. Dan and I disc
67 };
68
69 } // namespace WebCore
70
71 #endif // #ifndef MIDIAccessResolver_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698