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

Unified Diff: content/renderer/render_view_impl.cc

Issue 14247018: Implement WebRTC in Chrome for TV (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed some comments Created 7 years, 8 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_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index e0543f2b7d8afac9f35afd16e9e04bddb138b4e6..d526d3fb4992b77c700fe9faab1f56d729624c78 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -227,6 +227,12 @@
#include "webkit/media/android/media_player_bridge_manager_impl.h"
#include "webkit/media/android/webmediaplayer_android.h"
#include "webkit/media/android/webmediaplayer_manager_android.h"
+
+#if defined(GOOGLE_TV)
+#include "content/renderer/media/rtc_video_decoder_bridge_tv.h"
+#include "webkit/media/android/webmediaplayer_tv.h"
+#endif
+
#elif defined(OS_WIN)
// TODO(port): these files are currently Windows only because they concern:
// * theming
@@ -2690,7 +2696,7 @@ WebMediaPlayer* RenderViewImpl::createMediaPlayer(
RenderViewObserver, observers_, WillCreateMediaPlayer(frame, client));
const CommandLine* cmd_line = CommandLine::ForCurrentProcess();
-#if defined(ENABLE_WEBRTC)
+#if defined(ENABLE_WEBRTC) && !defined(GOOGLE_TV)
if (MediaStreamImpl::CheckMediaStream(url)) {
#if defined(OS_ANDROID) && defined(ARCH_CPU_ARMEL)
bool found_neon =
@@ -2723,15 +2729,40 @@ WebMediaPlayer* RenderViewImpl::createMediaPlayer(
media_player_proxy_ = new WebMediaPlayerProxyImplAndroid(
this, media_player_manager_.get());
}
- return new webkit_media::WebMediaPlayerAndroid(
+#if defined(GOOGLE_TV)
+ MediaStreamImpl* media_stream_impl = NULL;
+ media::Demuxer* demuxer = NULL;
+#if defined(ENABLE_WEBRTC)
+ if (MediaStreamImpl::CheckMediaStream(url)) {
+ EnsureMediaStreamImpl();
+ demuxer = RTCVideoDecoderBridgeTv::Get()->CreateDemuxer(
+ base::MessageLoopProxy::current());
+ if (demuxer)
+ media_stream_impl = media_stream_impl_;
+ else
+ return NULL;
ycheo (away) 2013/05/02 02:45:21 Could you add some error message for this?
wonsik 2013/05/02 15:22:00 Done.
+ }
+#endif // defined(ENABLE_WEBRTC)
+ return new webkit_media::WebMediaPlayerTv(
frame,
client,
media_player_manager_.get(),
media_player_proxy_,
new StreamTextureFactoryImpl(
context_provider->Context3d(), gpu_channel_host, routing_id_),
- new RenderMediaLog());
-#endif
+ new RenderMediaLog(),
+ media_stream_impl,
+ demuxer);
+#else // defined(GOOGLE_TV)
+ return new webkit_media::WebMediaPlayerAndroid(
+ frame,
+ client,
+ media_player_manager_.get(),
+ media_player_proxy_,
+ new StreamTextureFactoryImpl(
+ context_provider->Context3d(), gpu_channel_host, routing_id_));
+#endif // defined(GOOGLE_TV)
+#endif // defined(OS_ANDROID)
scoped_refptr<media::AudioRendererSink> sink;
if (!cmd_line->HasSwitch(switches::kDisableAudio)) {

Powered by Google App Engine
This is Rietveld 408576698