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

Unified Diff: media/base/mock_filters.h

Issue 1759123003: Revert of Extract state management from WebMediaPlayerImpl. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/BUILD.gn ('k') | media/base/mock_filters.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/mock_filters.h
diff --git a/media/base/mock_filters.h b/media/base/mock_filters.h
index 482f159c82aeb67c3200fc927f991936a943aa9e..368a542fbda6e5f266663f96c20dce79732de98f 100644
--- a/media/base/mock_filters.h
+++ b/media/base/mock_filters.h
@@ -18,7 +18,6 @@
#include "media/base/decoder_buffer.h"
#include "media/base/decryptor.h"
#include "media/base/demuxer.h"
-#include "media/base/pipeline.h"
#include "media/base/pipeline_status.h"
#include "media/base/renderer.h"
#include "media/base/text_track.h"
@@ -30,72 +29,6 @@
#include "testing/gmock/include/gmock/gmock.h"
namespace media {
-
-class MockPipeline : public Pipeline {
- public:
- MockPipeline();
- virtual ~MockPipeline();
-
- // Note: Start() and Resume() declarations are not actually overrides; they
- // take scoped_ptr* instead of scoped_ptr so that they can be mock methods.
- // Private stubs for Start() and Resume() implement the actual Pipeline
- // interface by forwarding to these mock methods.
- MOCK_METHOD10(Start,
- void(Demuxer*,
- scoped_ptr<Renderer>*,
- const base::Closure&,
- const PipelineStatusCB&,
- const PipelineStatusCB&,
- const PipelineMetadataCB&,
- const BufferingStateCB&,
- const base::Closure&,
- const AddTextTrackCB&,
- const base::Closure&));
- MOCK_METHOD1(Stop, void(const base::Closure&));
- MOCK_METHOD2(Seek, void(base::TimeDelta, const PipelineStatusCB&));
- MOCK_METHOD1(Suspend, void(const PipelineStatusCB&));
- MOCK_METHOD3(Resume,
- void(scoped_ptr<Renderer>*,
- base::TimeDelta,
- const PipelineStatusCB&));
-
- // TODO(sandersd): This should automatically return true between Start() and
- // Stop(). (Or better, remove it from the interface entirely.)
- MOCK_CONST_METHOD0(IsRunning, bool());
-
- // TODO(sandersd): These should be regular getters/setters.
- MOCK_CONST_METHOD0(GetPlaybackRate, double());
- MOCK_METHOD1(SetPlaybackRate, void(double));
- MOCK_CONST_METHOD0(GetVolume, float());
- MOCK_METHOD1(SetVolume, void(float));
-
- // TODO(sandersd): These should probably have setters too.
- MOCK_CONST_METHOD0(GetMediaTime, base::TimeDelta());
- MOCK_CONST_METHOD0(GetBufferedTimeRanges, Ranges<base::TimeDelta>());
- MOCK_CONST_METHOD0(GetMediaDuration, base::TimeDelta());
- MOCK_METHOD0(DidLoadingProgress, bool());
- MOCK_CONST_METHOD0(GetStatistics, PipelineStatistics());
-
- MOCK_METHOD2(SetCdm, void(CdmContext*, const CdmAttachedCB&));
-
- private:
- // Forwarding stubs (see comment above).
- void Start(Demuxer* demuxer,
- scoped_ptr<Renderer> renderer,
- const base::Closure& ended_cb,
- const PipelineStatusCB& error_cb,
- const PipelineStatusCB& seek_cb,
- const PipelineMetadataCB& metadata_cb,
- const BufferingStateCB& buffering_state_cb,
- const base::Closure& duration_change_cb,
- const AddTextTrackCB& add_text_track_cb,
- const base::Closure& waiting_for_decryption_key_cb) override;
- void Resume(scoped_ptr<Renderer> renderer,
- base::TimeDelta timestamp,
- const PipelineStatusCB& seek_cb) override;
-
- DISALLOW_COPY_AND_ASSIGN(MockPipeline);
-};
class MockDemuxer : public Demuxer {
public:
« no previous file with comments | « media/BUILD.gn ('k') | media/base/mock_filters.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698