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

Unified Diff: media/remoting/remote_renderer_impl.cc

Issue 2538853002: Media Remoting: Draw remoting interstitial on poster image. (Closed)
Patch Set: Addressed comments. Created 4 years 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: media/remoting/remote_renderer_impl.cc
diff --git a/media/remoting/remote_renderer_impl.cc b/media/remoting/remote_renderer_impl.cc
index 77fac89d7e2830952d9dc85900584182d5786530..b57f133e79de3cba7f80b3f43b90224c53889670 100644
--- a/media/remoting/remote_renderer_impl.cc
+++ b/media/remoting/remote_renderer_impl.cc
@@ -36,8 +36,7 @@ RemoteRendererImpl::RemoteRendererImpl(
rpc_broker_(remoting_renderer_controller_->GetRpcBroker()),
rpc_handle_(remoting::RpcBroker::GetUniqueHandle()),
remote_renderer_handle_(remoting::kInvalidHandle),
- interstitial_ui_(video_renderer_sink,
- remoting_renderer_controller->pipeline_metadata()),
+ video_renderer_sink_(video_renderer_sink),
weak_factory_(this) {
VLOG(2) << __FUNCTION__;
// The constructor is running on the main thread.
@@ -578,10 +577,10 @@ void RemoteRendererImpl::OnFatalError(PipelineStatus error) {
void RemoteRendererImpl::UpdateInterstitial() {
DCHECK(main_task_runner_->BelongsToCurrentThread());
+ if (!remoting_renderer_controller_)
+ return;
- interstitial_ui_.ShowInterstitial(
- remoting_renderer_controller_->remoting_source()->state() ==
- RemotingSessionState::SESSION_STARTED);
+ remoting_renderer_controller_->ShowInterstitial(video_renderer_sink_);
}
} // namespace media

Powered by Google App Engine
This is Rietveld 408576698