[Presentation API] Change connection to 'connected' if start a presentation with "https://www.googl…

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

 // 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.
 
 #include "modules/presentation/Presentation.h"
 
 #include "core/dom/Document.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/Settings.h"
 #include "modules/presentation/PresentationController.h"
 #include "modules/presentation/PresentationReceiver.h"
 #include "modules/presentation/PresentationRequest.h"
 #include "platform/weborigin/KURL.h"
 #include "wtf/Vector.h"
 
 namespace blink {
 
 Presentation::Presentation(LocalFrame* frame) : ContextClient(frame) {}
 
 // static
 Presentation* Presentation::create(LocalFrame* frame) {
   ASSERT(frame);
   Presentation* presentation = new Presentation(frame);
   PresentationController* controller = PresentationController::from(*frame);
   ASSERT(controller);
   controller->setPresentation(presentation);
+
+  if (frame->settings() && frame->settings()->getPresentationReceiver()) {

[mlamouri (slow - plz ping), 2017/04/10 12:50:28]

Maybe add a comment explaining that this need to be eagerly created in order to
have the receiver associated with the client?

[zhaobin, 2017/04/10 21:05:59]

Code removed.

+    presentation->m_receiver =
+        new PresentationReceiver(frame, controller->client());
+  }
   return presentation;
 }
 
 DEFINE_TRACE(Presentation) {
   visitor->trace(m_defaultRequest);
   visitor->trace(m_receiver);
   ContextClient::trace(visitor);
 }
 
 PresentationRequest* Presentation::defaultRequest() const {
@@ skipping 23 matching lines @@
   if (!m_receiver) {
     PresentationController* controller = PresentationController::from(*frame());
     auto* client = controller ? controller->client() : nullptr;
     m_receiver = new PresentationReceiver(frame(), client);
   }
 
   return m_receiver;
 }
 
 }  // namespace blink