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

Unified Diff: content/renderer/render_frame_impl.cc

Issue 2310753002: Media Remoting: Data/Control plumbing between renderer and Media Router. (Closed)
Patch Set: Updated/Moved TODO comment in render_frame_impl.cc. And REBASE. 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 32e04de5c8bab6bf8ad95ef6af54496d3bbe3a0e..7e65340851c7fdef9021a16b858e6f362fac50f2 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -2643,6 +2643,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(),
@@ -6145,6 +6150,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