Make ScriptPromiseResolver RefCountedWillBeRefCountedGarbageCollected.

Source/modules/webmidi/MIDIAccessInitializer.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MIDIAccessInitializer_h
 #define MIDIAccessInitializer_h
 
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "modules/webmidi/MIDIAccessor.h"
@@ skipping 21 matching lines @@
             : id(id)
             , manufacturer(manufacturer)
             , name(name)
             , type(type)
             , version(version)
             , isActive(isActive) { }
     };
 
     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();
 
     // MIDIAccessorClient
     virtual void didAddInputPort(const String& id, const String& manufacturer, const String& name, const String& version, bool isActive) override;
     virtual void didAddOutputPort(const String& id, const String& manufacturer, const String& name, const String& version, bool isActive) override;
     virtual void didSetInputPortState(unsigned portIndex, bool isActive) override;
     virtual void didSetOutputPortState(unsigned portIndex, bool isActive) override;
     virtual void didStartSession(bool success, const String& error, const String& message) override;
@@ skipping 11 matching lines @@
 
     OwnPtr<MIDIAccessor> m_accessor;
     bool m_requestSysex;
     Vector<PortDescriptor> m_portDescriptors;
 };
 
 } // namespace blink
 
 
 #endif // MIDIAccessInitializer_h