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

third_party/WebKit/Source/modules/presentation/PresentationController.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/PresentationController.h"
 
 #include "core/dom/Document.h"
 #include "core/frame/LocalFrame.h"
 #include "modules/presentation/PresentationConnection.h"
 #include "public/platform/modules/presentation/WebPresentationClient.h"
@@ skipping 130 matching lines @@
   m_connections.add(connection);
 }
 
 void PresentationController::contextDestroyed() {
   if (m_client) {
     m_client->setController(nullptr);
     m_client = nullptr;
   }
 }
 
+PresentationConnection* PresentationController::findExistingConnection(
+    const blink::WebVector<blink::WebURL>& presentationUrls,
+    const blink::WebString& presentationId) {
+  for (const auto& connection : m_connections) {
+    for (const auto& presentationUrl : presentationUrls) {
+      if (connection->getState() !=
+              WebPresentationConnectionState::Terminated &&
+          connection->matches(presentationId, presentationUrl))
+        return connection.get();

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

style: you need { } because the condition is more than one line.

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

Done.

+    }
+  }
+  return nullptr;
+}
+
 PresentationConnection* PresentationController::findConnection(
     WebPresentationConnectionClient* connectionClient) {
   for (const auto& connection : m_connections) {
     if (connection->matches(connectionClient))
       return connection.get();
   }
 
   return nullptr;
 }
 
 }  // namespace blink