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

Side by Side Diff: media/filters/renderer_impl.cc

Issue 827013005: Avoid double task trampoline for Pipeline state transitions. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Cleanup. Created 5 years, 11 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 unified diff | Download patch
« no previous file with comments | « media/filters/chunk_demuxer.cc ('k') | media/mojo/services/mojo_renderer_impl.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "media/filters/renderer_impl.h" 5 #include "media/filters/renderer_impl.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/callback.h" 8 #include "base/callback.h"
9 #include "base/callback_helpers.h" 9 #include "base/callback_helpers.h"
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
11 #include "base/location.h" 11 #include "base/location.h"
12 #include "base/single_thread_task_runner.h" 12 #include "base/single_thread_task_runner.h"
13 #include "media/base/audio_renderer.h" 13 #include "media/base/audio_renderer.h"
14 #include "media/base/bind_to_current_loop.h"
14 #include "media/base/demuxer_stream_provider.h" 15 #include "media/base/demuxer_stream_provider.h"
15 #include "media/base/time_source.h" 16 #include "media/base/time_source.h"
16 #include "media/base/video_renderer.h" 17 #include "media/base/video_renderer.h"
17 #include "media/base/wall_clock_time_source.h" 18 #include "media/base/wall_clock_time_source.h"
18 19
19 namespace media { 20 namespace media {
20 21
21 RendererImpl::RendererImpl( 22 RendererImpl::RendererImpl(
22 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner, 23 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner,
23 scoped_ptr<AudioRenderer> audio_renderer, 24 scoped_ptr<AudioRenderer> audio_renderer,
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 DCHECK(!error_cb.is_null()); 71 DCHECK(!error_cb.is_null());
71 DCHECK(demuxer_stream_provider->GetStream(DemuxerStream::AUDIO) || 72 DCHECK(demuxer_stream_provider->GetStream(DemuxerStream::AUDIO) ||
72 demuxer_stream_provider->GetStream(DemuxerStream::VIDEO)); 73 demuxer_stream_provider->GetStream(DemuxerStream::VIDEO));
73 74
74 demuxer_stream_provider_ = demuxer_stream_provider; 75 demuxer_stream_provider_ = demuxer_stream_provider;
75 statistics_cb_ = statistics_cb; 76 statistics_cb_ = statistics_cb;
76 buffering_state_cb_ = buffering_state_cb; 77 buffering_state_cb_ = buffering_state_cb;
77 paint_cb_ = paint_cb; 78 paint_cb_ = paint_cb;
78 ended_cb_ = ended_cb; 79 ended_cb_ = ended_cb;
79 error_cb_ = error_cb; 80 error_cb_ = error_cb;
81 init_cb_ = init_cb;
80 82
81 init_cb_ = init_cb;
82 state_ = STATE_INITIALIZING; 83 state_ = STATE_INITIALIZING;
83 InitializeAudioRenderer(); 84 InitializeAudioRenderer();
84 } 85 }
85 86
86 void RendererImpl::SetCdm(CdmContext* cdm_context, 87 void RendererImpl::SetCdm(CdmContext* cdm_context,
87 const CdmAttachedCB& cdm_attached_cb) { 88 const CdmAttachedCB& cdm_attached_cb) {
88 DVLOG(1) << __FUNCTION__; 89 DVLOG(1) << __FUNCTION__;
89 DCHECK(task_runner_->BelongsToCurrentThread()); 90 DCHECK(task_runner_->BelongsToCurrentThread());
90 DCHECK(cdm_context); 91 DCHECK(cdm_context);
91 92
(...skipping 468 matching lines...) Expand 10 before | Expand all | Expand 10 after
560 DCHECK(task_runner_->BelongsToCurrentThread()); 561 DCHECK(task_runner_->BelongsToCurrentThread());
561 562
562 if (!init_cb_.is_null()) 563 if (!init_cb_.is_null())
563 base::ResetAndReturn(&init_cb_).Run(); 564 base::ResetAndReturn(&init_cb_).Run();
564 565
565 if (!flush_cb_.is_null()) 566 if (!flush_cb_.is_null())
566 base::ResetAndReturn(&flush_cb_).Run(); 567 base::ResetAndReturn(&flush_cb_).Run();
567 } 568 }
568 569
569 } // namespace media 570 } // namespace media
OLDNEW
« no previous file with comments | « media/filters/chunk_demuxer.cc ('k') | media/mojo/services/mojo_renderer_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698