[Presentation API] (5th) (1-UA) integrate controller and receiver side for 1-UA messaging

chrome/browser/extensions/api/tab_capture/offscreen_tab.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 "chrome/browser/extensions/api/tab_capture/offscreen_tab.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "chrome/browser/extensions/api/tab_capture/tab_capture_registry.h"
+#include "chrome/browser/media/router/receiver_presentation_service_delegate_impl.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/web_contents_sizer.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/render_widget_host_view.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/web_preferences.h"
 #include "extensions/browser/extension_host.h"
 #include "extensions/browser/process_manager.h"
 
 using content::WebContents;
@@ skipping 87 matching lines @@
 
   // Set initial size, if specified.
   if (!initial_size.IsEmpty())
     ResizeWebContents(offscreen_tab_web_contents_.get(),
                       gfx::Rect(initial_size));
 
   // Mute audio output.  When tab capture starts, the audio will be
   // automatically unmuted, but will be captured into the MediaStream.
   offscreen_tab_web_contents_->SetAudioMuted(true);
 
-  // TODO(imcheng): If |optional_presentation_id| is not empty, register it with
-  // the PresentationRouter.  http://crbug.com/513859

[miu, 2017/02/02 21:17:34]

Please include 513859 on the BUG= line in this change description.

[zhaobin, 2017/02/03 03:22:54]

Done.

   if (!optional_presentation_id.empty()) {
-    NOTIMPLEMENTED()
-        << "Register with PresentationRouter, id=" << optional_presentation_id;
+    DVLOG(1) << " Register with ReceiverPresentationServiceDelegateImpl, "
+             << "[presentation_id]: " << optional_presentation_id;
+    // Register the offscreen tab as the receiver of the offscreen presentation.

[miu, 2017/02/02 21:17:34]

I think a more-accurate comment here would be something like:

  Create a ReceiverPSDImpl associated with the offscreen tab's WebContents. The
new instance will set up the necessary infrastructure to allow controlling peers
the ability to connect to the offscreen tab.

[zhaobin, 2017/02/03 03:22:54]

Done.

+    media_router::ReceiverPresentationServiceDelegateImpl::CreateForWebContents(

[miu, 2017/02/02 21:17:35]

I dove into the call points from here, and I wonder if there's a potential bug
in assumptions made about the Profiles (BrowserContexts):

1. The offscreen tab will be created from the tab capture extension API. The
extension that launches the tab will be using whatever Profile the MR Extension
was loaded under.

2. The offscreen tab will, of course, create its own incognito Profile based on
the extension's Profile.

3. Code downstream of here will then create an
OffscreenPresentationManagerFactory using the BrowserContextKeyedServiceFactory.
IIUC, the BCKSF will do a backwards-lookup from the offscreen tab's incognito
Profile to get/create a factory that will be associated with the Profile of the
MR Extension.

4. Therefore, will pages in other Profiles that want to start 1UA presentations
be unable to access the OffscreenPresentationManagerFactory to connect to the
offscreen tab?

I suggest you make sure to test using the presentation API 1UA functionality in
multiple-profile scenarios (e.g., browser started-up on one profile, but you can
switch to another profile and start a controlling page that starts a
presentation). A browsertest might be appropriate here.

[zhaobin, 2017/02/03 03:22:54]

reconnect() for 1-ua has not been implemented yet. Created crbug.com/688233 to
track this issue.

+        offscreen_tab_web_contents_.get(), optional_presentation_id);
 
     if (auto* render_view_host =
             offscreen_tab_web_contents_->GetRenderViewHost()) {
       auto web_prefs = render_view_host->GetWebkitPreferences();
       web_prefs.presentation_receiver = true;
       render_view_host->UpdateWebkitPreferences(web_prefs);
     }
   }
 
   // Navigate to the initial URL.
@@ skipping 238 matching lines @@
 
   // Schedule the timer to check again in a second.
   capture_poll_timer_.Start(
       FROM_HERE,
       base::TimeDelta::FromSeconds(kPollIntervalInSeconds),
       base::Bind(&OffscreenTab::DieIfContentCaptureEnded,
                  base::Unretained(this)));
 }
 
 }  // namespace extensions