diff options
| author | sandersd@chromium.org <sandersd@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-12 23:27:30 +0000 |
|---|---|---|
| committer | sandersd@chromium.org <sandersd@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-12 23:27:30 +0000 |
| commit | 5d2b3e4cc61ddd72e737b5a3c88b7d2d33b1397b (patch) | |
| tree | b35bfc7e8464554fca45a47029e2074e55bea680 /media | |
| parent | fd99cdd8035ee2d493f2edef8b845594fef37fb9 (diff) | |
| download | chromium_src-5d2b3e4cc61ddd72e737b5a3c88b7d2d33b1397b.zip chromium_src-5d2b3e4cc61ddd72e737b5a3c88b7d2d33b1397b.tar.gz chromium_src-5d2b3e4cc61ddd72e737b5a3c88b7d2d33b1397b.tar.bz2 | |
Non-const impls. for WebMediaPlayer::didLoadingProgress().
This provides non-const didLoadingProgress() implementations for WebMediaPlayer subclasses in Chromium so that the interface can switch to being non-const. It also removes the const specifier from DidLoadingProgress() implementations (Pipeline and DemuxerHost) and the mutable specifier for their associated properties.
This is the first of three changes; the second is
https://codereview.chromium.org/272223004.
BUG=360251
Review URL: https://codereview.chromium.org/275303002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269910 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media')
| -rw-r--r-- | media/base/pipeline.cc | 2 | ||||
| -rw-r--r-- | media/base/pipeline.h | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/media/base/pipeline.cc b/media/base/pipeline.cc index 841a49b..90ce799 100644 --- a/media/base/pipeline.cc +++ b/media/base/pipeline.cc @@ -169,7 +169,7 @@ TimeDelta Pipeline::GetMediaDuration() const { return clock_->Duration(); } -bool Pipeline::DidLoadingProgress() const { +bool Pipeline::DidLoadingProgress() { base::AutoLock auto_lock(lock_); bool ret = did_loading_progress_; did_loading_progress_ = false; diff --git a/media/base/pipeline.h b/media/base/pipeline.h index 1464cc2..b40a660 100644 --- a/media/base/pipeline.h +++ b/media/base/pipeline.h @@ -172,7 +172,7 @@ class MEDIA_EXPORT Pipeline : public DemuxerHost { // Return true if loading progress has been made since the last time this // method was called. - bool DidLoadingProgress() const; + bool DidLoadingProgress(); // Gets the current pipeline statistics. PipelineStatistics GetStatistics() const; @@ -345,7 +345,7 @@ class MEDIA_EXPORT Pipeline : public DemuxerHost { // True when AddBufferedTimeRange() has been called more recently than // DidLoadingProgress(). - mutable bool did_loading_progress_; + bool did_loading_progress_; // Current volume level (from 0.0f to 1.0f). This value is set immediately // via SetVolume() and a task is dispatched on the task runner to notify the |