diff options
| author | acolwell@chromium.org <acolwell@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-16 17:14:25 +0000 |
|---|---|---|
| committer | acolwell@chromium.org <acolwell@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-16 17:14:25 +0000 |
| commit | 236119cf32eb61f4b775ba573ecf46a1c504e7a3 (patch) | |
| tree | 9a292fdefecf37aa48d448f3f7706ab5521e4d33 /media/base/demuxer.cc | |
| parent | 7046bfcb6c8f46d0693c129d52de27bc68042af7 (diff) | |
| download | chromium_src-236119cf32eb61f4b775ba573ecf46a1c504e7a3.zip chromium_src-236119cf32eb61f4b775ba573ecf46a1c504e7a3.tar.gz chromium_src-236119cf32eb61f4b775ba573ecf46a1c504e7a3.tar.bz2 | |
Removing DataSource from Filter hierarchy and refactoring FilterHost into DemuxerHost & DataSourceHost.
Over the last year, several refactorings have caused DataSource to have almost nothing in common with the Filter interface. This change removes it from the Filter class hierarchy and splits up the FilterHost interface so that DataSource, Demuxer, and Filter have their own host interfaces. Splitting FilterHost improves encapsulation and makes it easier to reason about which host methods are required by different parts of the code.
BUG=
TEST=media_unittests
Review URL: http://codereview.chromium.org/8936014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114819 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media/base/demuxer.cc')
| -rw-r--r-- | media/base/demuxer.cc | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/media/base/demuxer.cc b/media/base/demuxer.cc index 83bfce3..b6b2d18 100644 --- a/media/base/demuxer.cc +++ b/media/base/demuxer.cc @@ -8,11 +8,13 @@ namespace media { +DemuxerHost::~DemuxerHost() {} + Demuxer::Demuxer() : host_(NULL) {} Demuxer::~Demuxer() {} -void Demuxer::set_host(FilterHost* host) { +void Demuxer::set_host(DemuxerHost* host) { DCHECK(host); DCHECK(!host_); host_ = host; |