Make ScriptPromiseResolver RefCountedWillBeRefCountedGarbageCollected.

Source/modules/webmidi/MIDIAccessInitializer.h

Index: Source/modules/webmidi/MIDIAccessInitializer.h
diff --git a/Source/modules/webmidi/MIDIAccessInitializer.h b/Source/modules/webmidi/MIDIAccessInitializer.h
index 0babda9d10b65dd7082a84c422b94a7b776d9679..6a26940101cb8511da0c1b92cc19ec0ab94b30c4 100644
--- a/Source/modules/webmidi/MIDIAccessInitializer.h
+++ b/Source/modules/webmidi/MIDIAccessInitializer.h
@@ -39,10 +39,10 @@ public:
 
     static ScriptPromise start(ScriptState* scriptState, const MIDIOptions& options)
     {
-        RefPtr<MIDIAccessInitializer> p = adoptRef(new MIDIAccessInitializer(scriptState, options));
-        p->keepAliveWhilePending();
-        p->suspendIfNeeded();
-        return p->start();
+        RefPtrWillBeRawPtr<MIDIAccessInitializer> resolver = adoptRefWillBeNoop(new MIDIAccessInitializer(scriptState, options));

[sof, 2014/12/16 21:34:59]

Not directly overlapping with this change, but how about moving this to the
implementation file at the same time? It seems like too much code to have in a
header file for no particular reason (and why isn't it called create()?)

[sof, 2014/12/16 22:07:05]

This block of code repeats in a number of create() methods in this CL - possible
to share via a function template? (not for this CL though.)

[tasak, 2014/12/17 08:40:08]

I see. I will talk with yhirano@.

+        resolver->keepAliveWhilePending();
+        resolver->suspendIfNeeded();
+        return resolver->start();
     }
 
     virtual ~MIDIAccessInitializer();