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

content/browser/presentation/presentation_service_impl.cc

 // 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 "content/browser/presentation/presentation_service_impl.h"
 
 #include <stddef.h>
 #include <stdint.h>
 #include <algorithm>
 #include <utility>
@@ skipping 399 matching lines @@
   std::string curr_url_host = navigation_handle->GetURL().host();
 
   // If a frame navigation is same-document (e.g. navigating to a fragment in
   // same page) then we do not unregister listeners.
   DVLOG(2) << "DidNavigateAnyFrame: "
            << "prev host: " << prev_url_host << ", curr host: " << curr_url_host
            << ", is_same_document: " << navigation_handle->IsSameDocument();
   if (navigation_handle->IsSameDocument())
     return;
 
+  // If a frame is receiver frame and it navigates from blank to some url (e.g.
+  // https://www.google.com), we do not unregister the corresponding offscreen
+  // presentation.
+  if (receiver_delegate_)

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

How do we know the navigation came from blank here?

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

Code removed.

+    return;
+
   // Reset if the frame actually navigated.
   Reset();
 }
 
 void PresentationServiceImpl::RenderFrameDeleted(
     content::RenderFrameHost* render_frame_host) {
   DVLOG(2) << "PresentationServiceImpl::RenderFrameDeleted";
   if (!FrameMatches(render_frame_host))
     return;
 
@@ skipping 80 matching lines @@
 
 void PresentationServiceImpl::NewPresentationCallbackWrapper::Run(
     const base::Optional<PresentationInfo>& presentation_info,
     const base::Optional<PresentationError>& error) {
   DCHECK(!callback_.is_null());
   callback_.Run(presentation_info, error);
   callback_.Reset();
 }
 
 }  // namespace content