Add promise-based addIceCandidate, setLocalDescription and setRemoteDescription to RTCPeerConnection

third_party/WebKit/Source/modules/mediastream/RTCVoidRequestPromiseImpl.cpp

Index: third_party/WebKit/Source/modules/mediastream/RTCVoidRequestPromiseImpl.cpp
diff --git a/third_party/WebKit/Source/modules/mediastream/RTCVoidRequestPromiseImpl.cpp b/third_party/WebKit/Source/modules/mediastream/RTCVoidRequestPromiseImpl.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..dbe739a13bf6ff6062bc398e2e30bcdc7ded189e
--- /dev/null
+++ b/third_party/WebKit/Source/modules/mediastream/RTCVoidRequestPromiseImpl.cpp
@@ -0,0 +1,70 @@
+// Copyright 2016 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.
+
+#include "modules/mediastream/RTCVoidRequestPromiseImpl.h"
+
+#include "bindings/core/v8/ScriptPromiseResolver.h"
+#include "core/dom/DOMException.h"
+#include "modules/mediastream/RTCPeerConnection.h"
+
+namespace blink {
+
+RTCVoidRequestPromiseImpl* RTCVoidRequestPromiseImpl::create(RTCPeerConnection* requester, ScriptPromiseResolver* resolver, ExceptionCode exceptionCode)
+{
+    RTCVoidRequestPromiseImpl* request = new RTCVoidRequestPromiseImpl(requester, resolver, exceptionCode);
+    return request;

[haraken, 2016/02/13 15:07:18]

return new RTCVoidRequestPromiseImpl(...)

[Guido Urdaneta, 2016/02/15 16:56:51]

Done.

+}
+
+RTCVoidRequestPromiseImpl::RTCVoidRequestPromiseImpl(RTCPeerConnection* requester, ScriptPromiseResolver* resolver, ExceptionCode exceptionCode)
+    : m_requester(requester)
+    , m_resolver(resolver)
+    , m_exceptionCode(exceptionCode)
+{
+    ASSERT(m_requester);
+    ASSERT(m_resolver);
+}
+
+RTCVoidRequestPromiseImpl::~RTCVoidRequestPromiseImpl()
+{
+}
+
+void RTCVoidRequestPromiseImpl::requestSucceeded()
+{
+    if (m_requester && m_requester->shouldFireDefaultCallbacks()) {
+        m_resolver->resolve();
+    } else {
+        // This is needed to have the resolver release its internal resources
+        // while leaving the associated promise pending as specified.
+        m_resolver->stop();

[haraken, 2016/02/13 15:07:18]

ActiveDOMObject::resume/suspend/stop should not be called by other than
LifecycleNotifer.

yhirano@: Would it be ok to make ScriptPromiseResolver::clear a public method
and call clear here?

[yhirano, 2016/02/13 16:13:37]

Yes. You need to stop the timer as well as in stop().

[Guido Urdaneta, 2016/02/15 16:56:51]

What should I do? Stay with stop() for now or wait until clear() is public?

+    }
+
+    clear();
+}
+
+void RTCVoidRequestPromiseImpl::requestFailed(const String& error)
+{
+    if (m_requester && m_requester->shouldFireDefaultCallbacks()) {
+        m_resolver->reject(DOMException::create(m_exceptionCode, error));
+    } else {
+        // This is needed to have the resolver release its internal resources
+        // while leaving the associated promise pending as specified.
+        m_resolver->stop();
+    }
+
+    clear();
+}
+
+void RTCVoidRequestPromiseImpl::clear()
+{
+    m_requester.clear();
+}
+
+DEFINE_TRACE(RTCVoidRequestPromiseImpl)
+{
+    visitor->trace(m_resolver);
+    visitor->trace(m_requester);
+    RTCVoidRequest::trace(visitor);
+}
+
+} // namespace blink