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

Side by Side Diff: chromecast/media/service/cast_renderer.cc

Issue 2541803003: media: remove unused functions: Renderer::HasAudio()/HasVideo() (Closed)
Patch Set: cast 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chromecast/media/service/cast_renderer.h" 5 #include "chromecast/media/service/cast_renderer.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/single_thread_task_runner.h" 8 #include "base/single_thread_task_runner.h"
9 #include "chromecast/base/task_runner_impl.h" 9 #include "chromecast/base/task_runner_impl.h"
10 #include "chromecast/media/base/video_resolution_policy.h" 10 #include "chromecast/media/base/video_resolution_policy.h"
(...skipping 191 matching lines...) Expand 10 before | Expand all | Expand 10 after
202 void CastRenderer::SetVolume(float volume) { 202 void CastRenderer::SetVolume(float volume) {
203 DCHECK(task_runner_->BelongsToCurrentThread()); 203 DCHECK(task_runner_->BelongsToCurrentThread());
204 pipeline_->SetVolume(volume); 204 pipeline_->SetVolume(volume);
205 } 205 }
206 206
207 base::TimeDelta CastRenderer::GetMediaTime() { 207 base::TimeDelta CastRenderer::GetMediaTime() {
208 DCHECK(task_runner_->BelongsToCurrentThread()); 208 DCHECK(task_runner_->BelongsToCurrentThread());
209 return pipeline_->GetMediaTime(); 209 return pipeline_->GetMediaTime();
210 } 210 }
211 211
212 bool CastRenderer::HasAudio() {
213 DCHECK(task_runner_->BelongsToCurrentThread());
214 return pipeline_->HasAudio();
215 }
216
217 bool CastRenderer::HasVideo() {
218 DCHECK(task_runner_->BelongsToCurrentThread());
219 return pipeline_->HasVideo();
220 }
221
222 void CastRenderer::OnVideoResolutionPolicyChanged() { 212 void CastRenderer::OnVideoResolutionPolicyChanged() {
223 DCHECK(task_runner_->BelongsToCurrentThread()); 213 DCHECK(task_runner_->BelongsToCurrentThread());
224 if (!video_resolution_policy_) 214 if (!video_resolution_policy_)
225 return; 215 return;
226 216
227 if (video_resolution_policy_->ShouldBlock(video_res_)) 217 if (video_resolution_policy_->ShouldBlock(video_res_))
228 OnError(::media::PIPELINE_ERROR_DECODE); 218 OnError(::media::PIPELINE_ERROR_DECODE);
229 } 219 }
230 220
231 void CastRenderer::OnError(::media::PipelineStatus status) { 221 void CastRenderer::OnError(::media::PipelineStatus status) {
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
268 } 258 }
269 259
270 void CastRenderer::OnVideoOpacityChange(bool opaque) { 260 void CastRenderer::OnVideoOpacityChange(bool opaque) {
271 DCHECK(task_runner_->BelongsToCurrentThread()); 261 DCHECK(task_runner_->BelongsToCurrentThread());
272 DCHECK(opaque); 262 DCHECK(opaque);
273 client_->OnVideoOpacityChange(opaque); 263 client_->OnVideoOpacityChange(opaque);
274 } 264 }
275 265
276 } // namespace media 266 } // namespace media
277 } // namespace chromecast 267 } // namespace chromecast
OLDNEW
« no previous file with comments | « chromecast/media/service/cast_renderer.h ('k') | content/browser/media/android/media_player_renderer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698