[Presentation API] Resolve PresentationRequest::reconnect() with existing presentation connection i…

third_party/WebKit/Source/modules/presentation/PresentationConnectionCallbacks.cpp

 // Copyright 2015 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/presentation/PresentationConnectionCallbacks.h"
 
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "core/dom/DOMException.h"
 #include "modules/presentation/PresentationConnection.h"
 #include "modules/presentation/PresentationError.h"
@@ skipping 27 matching lines @@
 }
 
 void PresentationConnectionCallbacks::onError(
     const WebPresentationError& error) {
   if (!m_resolver->getExecutionContext() ||
       m_resolver->getExecutionContext()->isContextDestroyed())
     return;
   m_resolver->reject(PresentationError::take(error));
 }
 
+ExistingPresentationConnectionCallbacks::
+    ExistingPresentationConnectionCallbacks(ScriptPromiseResolver* resolver,
+                                            PresentationConnection* connection)
+    : m_resolver(resolver), m_connection(connection) {
+  DCHECK(m_resolver);
+  DCHECK(m_connection);
+}
+
+void ExistingPresentationConnectionCallbacks::onSuccess(
+    std::unique_ptr<WebPresentationConnectionClient>
+        PresentationConnectionClient) {
+  if (!m_resolver->getExecutionContext() ||
+      m_resolver->getExecutionContext()->isContextDestroyed())
+    return;

[mlamouri (slow - plz ping), 2017/01/03 12:02:11]

style: add { } because of 2-line condition. You can also update the code above.

[zhaobin, 2017/01/03 20:12:07]

Done.

+
+  if (m_connection->getState() == WebPresentationConnectionState::Closed)
+    m_connection->didChangeState(WebPresentationConnectionState::Connecting);
+
+  m_resolver->resolve(m_connection);
+}
+
+void ExistingPresentationConnectionCallbacks::onError(
+    const WebPresentationError& error) {
+  NOTREACHED();
+}
+
 }  // namespace blink