[Presentation API] (1st) (1-UA) Add is_offscreen_presentation attribute to MediaRouter

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 d0c574cfd30e5b0d9beee443660035d56d5e9618..7d9400ca68572e5e1bc8aa244132da7b911a89e9 100644
--- a/chrome/browser/media/router/presentation_service_delegate_impl.cc
+++ b/chrome/browser/media/router/presentation_service_delegate_impl.cc
@@ -756,11 +756,13 @@ void PresentationServiceDelegateImpl::OnStartSessionSucceeded(
     int render_frame_id,
     const content::PresentationSessionStartedCallback& success_cb,
     const content::PresentationSessionInfo& new_session,
-    const MediaRoute::Id& route_id) {
+    const MediaRoute& route) {
+  const MediaRoute::Id& route_id = route.media_route_id();
   DVLOG(1) << "OnStartSessionSucceeded: "
            << "route_id: " << route_id
            << ", presentation URL: " << new_session.presentation_url
-           << ", presentation ID: " << new_session.presentation_id;
+           << ", presentation ID: " << new_session.presentation_id
+           << ", is_offscreen_presentation: " << new_session.is_offscreen;
   frame_manager_->OnPresentationSessionStarted(
       RenderFrameHostId(render_process_id, render_frame_id), new_session,
       route_id);

[mark a. foltz, 2016/11/03 18:32:05]

Would be nice to pass |route| here instead of |route_id|.  Optional cleanup to
do in this patch.

[zhaobin, 2016/11/03 23:00:46]

Will do it in a seperate CL. |route_id| used in lots of place, create a bug to
track this: https://bugs.chromium.org/p/chromium/issues/detail?id=662208#