[Presentation API] (1-UA) fire PresentationConnection onterminate event if receiver page gets destr…

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 539 matching lines @@
   Reset();
   delete this;
 }
 
 void PresentationServiceImpl::Reset() {
   DVLOG(2) << "PresentationServiceImpl::Reset";
 
   if (auto* delegate = GetPresentationServiceDelegate())
     delegate->Reset(render_process_id_, render_frame_id_);
 
+  if (receiver_delegate_) {
+    PresentationSessionInfo dummy_session(GURL("fake_url"), "fake_id");

[mark a. foltz, 2017/02/25 01:43:12]

This is another case where having separate mojoms for the controller and
receiver would make things clearer (no need to pass a dummy connection).  Can
you add a TODO to remove this when the refactoring is done?

[zhaobin, 2017/02/28 04:25:19]

Removed :)

+    OnConnectionStateChanged(dummy_session,
+                             PresentationConnectionStateChangeInfo(
+                                 PRESENTATION_CONNECTION_STATE_TERMINATED));
+  }
+
   default_presentation_urls_.clear();
 
   screen_availability_listeners_.clear();
 
   start_session_request_id_ = kInvalidRequestSessionId;
   pending_start_session_cb_.reset();
 
   pending_join_session_cbs_.clear();
 
   if (on_connection_messages_callback_.get()) {
@@ skipping 64 matching lines @@
 
 void PresentationServiceImpl::NewSessionCallbackWrapper::Run(
     const base::Optional<PresentationSessionInfo>& session_info,
     const base::Optional<PresentationError>& error) {
   DCHECK(!callback_.is_null());
   callback_.Run(session_info, error);
   callback_.Reset();
 }
 
 }  // namespace content