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

third_party/WebKit/Source/modules/presentation/NavigatorPresentation.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/NavigatorPresentation.h"
 
+#include "core/dom/Document.h"
+#include "core/frame/LocalDOMWindow.h"
+#include "core/frame/LocalFrame.h"
 #include "core/frame/Navigator.h"
 #include "modules/presentation/Presentation.h"
 
 namespace blink {
 
 NavigatorPresentation::NavigatorPresentation() {}
 
 // static
 const char* NavigatorPresentation::supplementName() {
   return "NavigatorPresentation";
 }
 
 // static
+NavigatorPresentation* NavigatorPresentation::from(Document& document) {
+  if (!document.frame() || !document.frame()->domWindow())
+    return 0;
+  Navigator& navigator = *document.frame()->domWindow()->navigator();
+  return &from(navigator);
+}
+
+// static
 NavigatorPresentation& NavigatorPresentation::from(Navigator& navigator) {
   NavigatorPresentation* supplement = static_cast<NavigatorPresentation*>(
       Supplement<Navigator>::from(navigator, supplementName()));
   if (!supplement) {
     supplement = new NavigatorPresentation();
+    if (navigator.frame())
+      supplement->m_presentation = Presentation::create(navigator.frame());

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

This seems to duplicate the logic in `NavigatorPresentation::presentation()`. Do
you do this so when LocalFrameClientImpl gets the NavigatorPresentation, it will
create the receiver object? Would it make sense to have a method that make this
clear such as `ensurePresentationReceiver`?

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

Done.

+
     provideTo(navigator, supplementName(), supplement);
   }
   return *supplement;
 }
 
 // static
 Presentation* NavigatorPresentation::presentation(Navigator& navigator) {
   NavigatorPresentation& self = NavigatorPresentation::from(navigator);
   if (!self.m_presentation) {
     if (!navigator.frame())
       return nullptr;
     self.m_presentation = Presentation::create(navigator.frame());
   }
   return self.m_presentation;
 }
 
 DEFINE_TRACE(NavigatorPresentation) {
   visitor->trace(m_presentation);
   Supplement<Navigator>::trace(visitor);
 }
 
 }  // namespace blink