[Presentation API] (MR side) 1-UA: notify receiver page when receiver connection becomes available

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/profiles/profile.h"
 #include "chrome/browser/ui/web_contents_sizer.h"
 #include "content/public/browser/render_widget_host_view.h"
 #include "content/public/browser/web_contents.h"
 #include "extensions/browser/extension_host.h"
 #include "extensions/browser/process_manager.h"
 
+#if defined(ENABLE_MEDIA_ROUTER)

[imcheng, 2016/09/19 23:01:17]

No need for the ENABLE_MEDIA_ROUTER check. We should just get rid of it.

[zhaobin, 2016/09/23 17:18:18]

Done.

+#include "chrome/browser/media/router/media_router_feature.h"

[imcheng, 2016/09/19 23:01:17]

this include not needed?

[zhaobin, 2016/09/23 17:18:18]

Done.

+#include "chrome/browser/media/router/receiver_presentation_service_delegate_impl.h"
+#endif
+
 using content::WebContents;
 
 DEFINE_WEB_CONTENTS_USER_DATA_KEY(extensions::OffscreenTabsOwner);
 
 namespace {
 
 // Upper limit on the number of simultaneous off-screen tabs per extension
 // instance.
 const int kMaxOffscreenTabsPerExtension = 3;
 
@@ skipping 81 matching lines @@
     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
   if (!optional_presentation_id.empty()) {
-    NOTIMPLEMENTED()
-        << "Register with PresentationRouter, id=" << optional_presentation_id;
+#if defined(ENABLE_MEDIA_ROUTER)
+    DVLOG(1) << "Register with PresentationRouter, id="

[imcheng, 2016/09/19 23:01:17]

I suggest just combining this log statement with the DVLOG above.

[zhaobin, 2016/09/23 17:18:18]

Done.

+             << optional_presentation_id;
+    // Register the offscreen tab as the receiver of the offscreen presentation.
+    media_router::ReceiverPresentationServiceDelegateImpl::CreateForWebContents(
+        offscreen_tab_web_contents_.get(), optional_presentation_id);
+#endif  // defined(ENABLE_MEDIA_ROUTER)
   }
 
   // Navigate to the initial URL.
   content::NavigationController::LoadURLParams load_params(start_url_);
   load_params.should_replace_current_entry = true;
   load_params.should_clear_history_list = true;
   offscreen_tab_web_contents_->GetController().LoadURLWithParams(load_params);
 
   start_time_ = base::TimeTicks::Now();
   DieIfContentCaptureEnded();
@@ skipping 229 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