[Presentation API] 1-UA presentation support + presenter APIs.

chrome/browser/chrome_content_browser_client.cc

 // Copyright (c) 2012 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/chrome_content_browser_client.h"
 
 #include <map>
 #include <set>
 #include <utility>
 #include <vector>
@@ skipping 247 matching lines @@
 #if defined(USE_BROWSER_SPELLCHECKER)
 #include "chrome/browser/spellchecker/spellcheck_message_filter_platform.h"
 #endif
 
 #if defined(ENABLE_WEBRTC)
 #include "chrome/browser/media/webrtc_logging_handler_host.h"
 #endif
 
 #if defined(ENABLE_MEDIA_ROUTER)
 #include "chrome/browser/media/router/presentation_service_delegate_impl.h"
+#include "chrome/browser/media/router/receiver_presentation_service_delegate_impl.h"
 #endif
 
 using base::FileDescriptor;
 using blink::WebWindowFeatures;
 using content::AccessTokenStore;
 using content::BrowserThread;
 using content::BrowserURLHandler;
 using content::ChildProcessSecurityPolicy;
 using content::QuotaPermissionContext;
 using content::RenderFrameHost;
@@ skipping 2309 matching lines @@
       browser_context, params, callback);
 #else
   NOTIMPLEMENTED();
 #endif
 }
 
 content::PresentationServiceDelegate*
 ChromeContentBrowserClient::GetPresentationServiceDelegate(
       content::WebContents* web_contents) {
 #if defined(ENABLE_MEDIA_ROUTER)
-  if (switches::MediaRouterEnabled() &&
-      !web_contents->GetBrowserContext()->IsOffTheRecord()) {
-    return media_router::PresentationServiceDelegateImpl::
-        GetOrCreateForWebContents(web_contents);
+  if (switches::MediaRouterEnabled()) {
+    if (web_contents->GetBrowserContext()->IsOffTheRecord()) {
+      return media_router::ReceiverPresentationServiceDelegateImpl::

[miu, 2015/10/07 21:51:46]

This is a little confusing.  How is ReceiverPresentationServiceDelegateImpl
appropriate for all off-the-record profiles?  Doesn't it only pertain to
off-screen tabs?

Perhaps this code should be:

  if (auto* impl = ReceiverPresServiceDelegateImpl::FromWebContents(...))
    return impl;  // WebContents is an off-screen tab.
  return PresentationServiceDelegateImpl::GetOrCreateForWebContents(...);

Also, OOC, should we be worried about extension background pages attempting to
use the navigator.presentation API directly?  Do we want to allow or forbid
that?

[imcheng, 2015/10/10 04:39:43]

We've disabled presentation API in incognito mode, so this method should return
null for non-offscreen incognito WebContents. I will restructure this and add
some comments so it's clearer.

+          FromWebContents(web_contents);
+    } else {
+      return media_router::PresentationServiceDelegateImpl::
+          GetOrCreateForWebContents(web_contents);
+    }
   }
 #endif
   return nullptr;
 }
 
 void ChromeContentBrowserClient::RecordURLMetric(const std::string& metric,
                                                  const GURL& url) {
   if (url.is_valid()) {
     rappor::SampleDomainAndRegistryFromGURL(g_browser_process->rappor_service(),
                                             metric, url);
@@ skipping 99 matching lines @@
   if (channel <= kMaxDisableEncryptionChannel) {
     static const char* const kWebRtcDevSwitchNames[] = {
       switches::kDisableWebRtcEncryption,
     };
     to_command_line->CopySwitchesFrom(from_command_line,
                                       kWebRtcDevSwitchNames,
                                       arraysize(kWebRtcDevSwitchNames));
   }
 }
 #endif  // defined(ENABLE_WEBRTC)