Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(81)

Unified Diff: content/renderer/render_frame_impl.cc

Issue 2310753002: Media Remoting: Data/Control plumbing between renderer and Media Router. (Closed)
Patch Set: Just a REBASE on ToT before commit. Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 0483c3d0781c3c8393f1c3d29009769c38df48bd..0140be77e41fd1d2f0907ae065c14d8d6281d22a 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -2700,6 +2700,11 @@ blink::WebMediaPlayer* RenderFrameImpl::createMediaPlayer(
if (!url_index_.get() || url_index_->frame() != frame_)
url_index_.reset(new media::UrlIndex(frame_));
+ // TODO(miu): In a soon-upcoming change, call GetRemoterFactory()->Create() to
+ // allow the local media pipeline to receive notifications about when Media
+ // Remoting can take place. Control logic in/around WebMediaPlayerImpl will
+ // implement media.mojom.RemotingSource. http://crbug.com/643964
+
media::WebMediaPlayerImpl* media_player = new media::WebMediaPlayerImpl(
frame_, client, encrypted_client,
GetWebMediaPlayerDelegate()->AsWeakPtr(),
@@ -6210,6 +6215,12 @@ bool RenderFrameImpl::AreSecureCodecsSupported() {
#endif // defined(OS_ANDROID)
}
+media::mojom::RemoterFactory* RenderFrameImpl::GetRemoterFactory() {
+ if (!remoter_factory_)
+ GetRemoteInterfaces()->GetInterface(&remoter_factory_);
+ return remoter_factory_.get();
+}
+
media::CdmFactory* RenderFrameImpl::GetCdmFactory() {
if (cdm_factory_)
return cdm_factory_.get();

Powered by Google App Engine
This is Rietveld 408576698