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

Unified Diff: media/filters/ffmpeg_video_decoder_unittest.cc

Issue 126160: Moving MockFFmpeg to its own file and some media.gyp cleanup. (Closed)
Patch Set: Not ready for mac and linux yet Created 11 years, 6 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/base/mock_ffmpeg.cc ('k') | media/media.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/ffmpeg_video_decoder_unittest.cc
diff --git a/media/filters/ffmpeg_video_decoder_unittest.cc b/media/filters/ffmpeg_video_decoder_unittest.cc
index 7cb0d7bcad29d757007e367676a1ff7c0e299161..4faf0f85814dc11dc2141d70adcf18fac324c37e 100644
--- a/media/filters/ffmpeg_video_decoder_unittest.cc
+++ b/media/filters/ffmpeg_video_decoder_unittest.cc
@@ -6,11 +6,11 @@
#include "base/singleton.h"
#include "media/base/filters.h"
+#include "media/base/mock_ffmpeg.h"
#include "media/base/mock_filter_host.h"
#include "media/filters/ffmpeg_common.h"
#include "media/filters/ffmpeg_interfaces.h"
#include "media/filters/ffmpeg_video_decoder.h"
-#include "testing/gmock/include/gmock/gmock.h"
#include "testing/gtest/include/gtest/gtest.h"
using ::testing::Return;
@@ -29,65 +29,8 @@ class MockDemuxerStream : public DemuxerStream, public AVStreamProvider {
MOCK_METHOD0(GetAVStream, AVStream*());
};
-class MockFFmpeg {
- public:
- MOCK_METHOD1(AVCodecFindDecoder, AVCodec*(enum CodecID id));
- MOCK_METHOD2(AVCodecOpen, int(AVCodecContext* avctx, AVCodec* codec));
- MOCK_METHOD2(AVCodecThreadInit, int(AVCodecContext* avctx, int threads));
-
- // Setter/getter for the global instance of MockFFmpeg.
- static void set(MockFFmpeg* instance) {
- instance_ = instance;
- }
-
- static MockFFmpeg* get() {
- return instance_;
- }
-
- private:
- static MockFFmpeg* instance_;
-};
-
-MockFFmpeg* MockFFmpeg::instance_ = NULL;
-
} // namespace media
-// FFmpeg mocks to remove dependency on having the DLLs present.
-extern "C" {
-
-AVCodec* avcodec_find_decoder(enum CodecID id) {
- return media::MockFFmpeg::get()->AVCodecFindDecoder(id);
-}
-
-int avcodec_open(AVCodecContext* avctx, AVCodec* codec) {
- return media::MockFFmpeg::get()->AVCodecOpen(avctx, codec);
-}
-
-int avcodec_thread_init(AVCodecContext* avctx, int threads) {
- return media::MockFFmpeg::get()->AVCodecThreadInit(avctx, threads);
-}
-
-void avcodec_flush_buffers(AVCodecContext* avctx) {
- NOTREACHED();
-}
-
-AVFrame* avcodec_alloc_frame() {
- NOTREACHED();
- return NULL;
-}
-
-int avcodec_decode_video2(AVCodecContext* avctx, AVFrame* picture,
- int* got_picture_ptr, AVPacket* avpkt) {
- NOTREACHED();
- return 0;
-}
-
-void av_init_packet(AVPacket* pkt) {
- NOTREACHED();
-}
-
-} // extern "C"
-
namespace media {
// Fixture class to facilitate writing tests. Takes care of setting up the
« no previous file with comments | « media/base/mock_ffmpeg.cc ('k') | media/media.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698