diff options
| author | scherkus <scherkus@chromium.org> | 2014-09-09 16:13:38 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2014-09-09 23:15:34 +0000 |
| commit | ece894570c5f32f7f404f4c8da9ff8b9f03b75cd (patch) | |
| tree | f2efa9db4debb764f71e76efab57152bbdc7673b /media/base/pipeline.cc | |
| parent | a1bf3af9b614208729f14af603abd5e6b5bae698 (diff) | |
| download | chromium_src-ece894570c5f32f7f404f4c8da9ff8b9f03b75cd.zip chromium_src-ece894570c5f32f7f404f4c8da9ff8b9f03b75cd.tar.gz chromium_src-ece894570c5f32f7f404f4c8da9ff8b9f03b75cd.tar.bz2 | |
Switch to using media::TimeSource inside media::RendererImpl.
This finally replaces the time-updating-callback inside of AudioRenderer
and VideoRenderer with a model where clients can query the time. As a
result, many other bits of cleanup are now possible:
- Renderers no longer need to be aware of the media duration
- TimeDeltaInterpolator can be replaced with WallClockTimeSource
- We can separate time values used for syncing video vs. informational
BUG=370634
Review URL: https://codereview.chromium.org/534073002
Cr-Commit-Position: refs/heads/master@{#294029}
Diffstat (limited to 'media/base/pipeline.cc')
| -rw-r--r-- | media/base/pipeline.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/media/base/pipeline.cc b/media/base/pipeline.cc index 6e680d5..086dd96 100644 --- a/media/base/pipeline.cc +++ b/media/base/pipeline.cc @@ -690,8 +690,7 @@ void Pipeline::InitializeRenderer(const base::Closure& done_cb) { base::Bind(&Pipeline::OnUpdateStatistics, weak_this), base::Bind(&Pipeline::OnRendererEnded, weak_this), base::Bind(&Pipeline::OnError, weak_this), - base::Bind(&Pipeline::BufferingStateChanged, weak_this), - base::Bind(&Pipeline::GetMediaDuration, base::Unretained(this))); + base::Bind(&Pipeline::BufferingStateChanged, weak_this)); } void Pipeline::ReportMetadata() { |