[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 8c3bdab4e699585fcb73132dc6746bfb77ef09c2..77e2fd7c3cc170abf0fe322505a53a6ec9df5f03 100644
--- a/content/browser/presentation/presentation_service_impl.cc
+++ b/content/browser/presentation/presentation_service_impl.cc
@@ -577,6 +577,13 @@ void PresentationServiceImpl::Reset() {
   if (auto delegate = GetPresentationServiceDelegate())
     delegate->Reset(render_process_id_, render_frame_id_);
 
+  if (receiver_delegate_) {

[imcheng, 2016/11/28 22:12:00]

If the receiver frame is already being destroyed then isn't this unnecessary?

[zhaobin, 2017/02/16 00:42:30]

This function is called before destroying PresentationDispatcher and
PresentationReceiver. Adding state change here could invoke receiver
connection's onterminate handler...

[imcheng, 2017/02/24 19:20:33]

I see. Note that Reset() also gets called on navigation.
- In that case does the offscreen tab capture stop?
- Does the controller side get notified of state change?
- Should we allow the offscreen tab to navigate at all?

[imcheng, 2017/02/24 19:20:33]

It feels like we should be able to observe for this case in the renderer /
PresentationDispatcher and notify the connection object(s) directly without
going through Mojo.

[zhaobin, 2017/02/25 01:31:03]

Move state change to PresentationDispatcher.

Controller will get notified by MRP.

Setting window.location.href on receiver page does not work. There seems to be
some mechanism disable navigation from offscreen_tab.

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

The offscreen tab should be closed if it attempts top-level navigation.  (I can
check the spec for the exact wording)
No.  However on close reading of the spec we don't have a flag blocking this. 
Filed https://github.com/w3c/presentation-api/issues/414 to update the spec.

+    PresentationSessionInfo dummy_session(GURL("fake_url"), "fake_id", true);
+    OnConnectionStateChanged(dummy_session,
+                             PresentationConnectionStateChangeInfo(
+                                 PRESENTATION_CONNECTION_STATE_TERMINATED));
+  }
+
   default_presentation_urls_.clear();
 
   screen_availability_listeners_.clear();