Presentation API: implement renderer side of PresentationConnection.close().

chrome/browser/media/router/presentation_service_delegate_impl.cc

Index: chrome/browser/media/router/presentation_service_delegate_impl.cc
diff --git a/chrome/browser/media/router/presentation_service_delegate_impl.cc b/chrome/browser/media/router/presentation_service_delegate_impl.cc
index bc145d2d57daccb92b092bbb7ea530b914eeb863..0123d5024eed9c5b8f6333ef49f6fd173b35acbc 100644
--- a/chrome/browser/media/router/presentation_service_delegate_impl.cc
+++ b/chrome/browser/media/router/presentation_service_delegate_impl.cc
@@ -730,6 +730,21 @@ void PresentationServiceDelegateImpl::CloseSession(
     DVLOG(1) << "No active route for: " << presentation_id;
     return;
   }
+
+  // TODO(mfoltz): implement CloseSession(). Probably rename CloseRoute()?
+  NOTIMPLEMENTED();

[whywhat, 2015/11/20 06:26:49]

This can probably implemented similar to onPresentationConnectionDetached()? The
MR is notified that the MediaRoute is no longer attached to the page.

[mark a. foltz, 2015/11/20 17:47:17]

This should call router_->OnPresentationSessionDetached(route_id) to notify the
MRP that the connection is going away.  PresentationFrame::Reset() should
probably call CloseSession(route_id) as part of the frame-level cleanup to
create one code path for this.

Note that Derek just landed a patch last night that changes the APIs to this
file so you will want to rebase.

https://chromium.googlesource.com/chromium/src.git/+/f3e5a01a75bbdd97e5d218c4...

[mlamouri (slow - plz ping), 2015/11/27 11:48:47]

Rebased. I left the TODO. I will have a look in a follow-up but the intent of
this CL was to implement the renderer side, not implement the full feature.

+}
+
+void PresentationServiceDelegateImpl::TerminateSession(
+    int render_process_id,
+    int render_frame_id,
+    const std::string& presentation_id) {
+  const MediaRoute::Id& route_id = frame_manager_->GetRouteId(
+      RenderFrameHostId(render_process_id, render_frame_id), presentation_id);
+  if (route_id.empty()) {
+    DVLOG(1) << "No active route for: " << presentation_id;
+    return;
+  }
   router_->CloseRoute(route_id);
 }