[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;
+
+  if (m_connection->getState() == WebPresentationConnectionState::Closed)
+    m_connection->didChangeState(WebPresentationConnectionState::Connecting);
+
+  m_resolver->resolve(m_connection);
+}
+
+void ExistingPresentationConnectionCallbacks::onError(

[mark a. foltz, 2016/12/23 18:39:43]

Looking at the spec, I don't think there's an error case possible in the case
for checking an existing connection; it will always fall through to check for a
connection from another browsing context.  So this can be NOTREACHED().

[zhaobin, 2016/12/30 05:05:00]

Done.

+    const WebPresentationError& error) {
+  if (!m_resolver->getExecutionContext() ||
+      m_resolver->getExecutionContext()->isContextDestroyed())
+    return;
+  m_resolver->reject(PresentationError::take(error));
+}
+
 }  // namespace blink