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

content/browser/presentation/presentation_service_impl.cc

Index: content/browser/presentation/presentation_service_impl.cc
diff --git a/content/browser/presentation/presentation_service_impl.cc b/content/browser/presentation/presentation_service_impl.cc
index a7f9277425495cf896ebeef4e0b62b78a6cfe017..f0b722fe4490cb286304a708408b22e775f781fd 100644
--- a/content/browser/presentation/presentation_service_impl.cc
+++ b/content/browser/presentation/presentation_service_impl.cc
@@ -557,6 +557,13 @@ void 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();