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

Unified Diff: webrtc/media/engine/internaldecoderfactory_unittest.cc

Issue 2521203002: Move VideoDecoder::Create() logic to separate internal video decoder factory (Closed)
Patch Set: Add unittest Created 4 years 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
Index: webrtc/media/engine/internaldecoderfactory_unittest.cc
diff --git a/webrtc/media/engine/videodecodersoftwarefallbackwrapper.h b/webrtc/media/engine/internaldecoderfactory_unittest.cc
similarity index 20%
copy from webrtc/media/engine/videodecodersoftwarefallbackwrapper.h
copy to webrtc/media/engine/internaldecoderfactory_unittest.cc
index 8e1a97d8179ce0e5170fc894eab81cd09dc2191f..bf1ca50f240474e9266b8a8cde7a9723c61f6587 100644
--- a/webrtc/media/engine/videodecodersoftwarefallbackwrapper.h
+++ b/webrtc/media/engine/internaldecoderfactory_unittest.cc
@@ -8,54 +8,14 @@
* be found in the AUTHORS file in the root of the source tree.
*/
-#ifndef WEBRTC_MEDIA_ENGINE_VIDEODECODERSOFTWAREFALLBACKWRAPPER_H_
-#define WEBRTC_MEDIA_ENGINE_VIDEODECODERSOFTWAREFALLBACKWRAPPER_H_
+#include "webrtc/media/engine/internaldecoderfactory.h"
-#include <memory>
-#include <string>
+#include "webrtc/test/gtest.h"
-#include "webrtc/video_decoder.h"
-
-namespace webrtc {
-
-// Class used to wrap external VideoDecoders to provide a fallback option on
-// software decoding when a hardware decoder fails to decode a stream due to
-// hardware restrictions, such as max resolution.
-class VideoDecoderSoftwareFallbackWrapper : public webrtc::VideoDecoder {
- public:
- VideoDecoderSoftwareFallbackWrapper(VideoCodecType codec_type,
- VideoDecoder* decoder);
-
- int32_t InitDecode(const VideoCodec* codec_settings,
- int32_t number_of_cores) override;
-
- int32_t Decode(const EncodedImage& input_image,
- bool missing_frames,
- const RTPFragmentationHeader* fragmentation,
- const CodecSpecificInfo* codec_specific_info,
- int64_t render_time_ms) override;
-
- int32_t RegisterDecodeCompleteCallback(
- DecodedImageCallback* callback) override;
-
- int32_t Release() override;
- bool PrefersLateDecoding() const override;
-
- const char* ImplementationName() const override;
-
- private:
- bool InitFallbackDecoder();
-
- const DecoderType decoder_type_;
- VideoDecoder* const decoder_;
-
- VideoCodec codec_settings_;
- int32_t number_of_cores_;
- std::string fallback_implementation_name_;
- std::unique_ptr<VideoDecoder> fallback_decoder_;
- DecodedImageCallback* callback_;
-};
-
-} // namespace webrtc
-
-#endif // WEBRTC_MEDIA_ENGINE_VIDEODECODERSOFTWAREFALLBACKWRAPPER_H_
+TEST(InternalDecoderFactory, TestVP8) {
+ cricket::InternalDecoderFactory factory;
+ webrtc::VideoDecoder* decoder =
+ factory.CreateVideoDecoder(webrtc::kVideoCodecVP8);
+ EXPECT_TRUE(decoder);
+ factory.DestroyVideoDecoder(decoder);
+}
« no previous file with comments | « webrtc/media/engine/internaldecoderfactory.cc ('k') | webrtc/media/engine/videodecodersoftwarefallbackwrapper.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698