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

Unified Diff: content/common/gpu/media/video_encode_accelerator_unittest.cc

Issue 1801343002: Revert of H264 HW encode using VideoToolbox (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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
Index: content/common/gpu/media/video_encode_accelerator_unittest.cc
diff --git a/content/common/gpu/media/video_encode_accelerator_unittest.cc b/content/common/gpu/media/video_encode_accelerator_unittest.cc
index b108ceb437e9fbda7dead882f13caf80b2747e6a..d0a4981ec449f8cb1a82a518a0566d09f21bc7cb 100644
--- a/content/common/gpu/media/video_encode_accelerator_unittest.cc
+++ b/content/common/gpu/media/video_encode_accelerator_unittest.cc
@@ -55,8 +55,6 @@
// Status has been defined as int in Xlib.h.
#undef Status
#endif // defined(ARCH_CPU_X86_FAMILY)
-#elif defined(OS_MACOSX)
-#include "content/common/gpu/media/vt_video_encode_accelerator_mac.h"
#else
#error The VideoEncodeAcceleratorUnittest is not supported on this platform.
#endif
@@ -128,11 +126,7 @@
// of the stream.
// Bitrate is only forced for tests that test bitrate.
const char* g_default_in_filename = "bear_320x192_40frames.yuv";
-#if !defined(OS_MACOSX)
const char* g_default_in_parameters = ":320:192:1:out.h264:200000";
-#else
-const char* g_default_in_parameters = ":320:192:0:out.h264:200000";
-#endif
// Enabled by including a --fake_encoder flag to the command line invoking the
// test.
@@ -818,7 +812,6 @@
scoped_ptr<media::VideoEncodeAccelerator> CreateFakeVEA();
scoped_ptr<media::VideoEncodeAccelerator> CreateV4L2VEA();
scoped_ptr<media::VideoEncodeAccelerator> CreateVaapiVEA();
- scoped_ptr<media::VideoEncodeAccelerator> CreateVTVEA();
void SetState(ClientState new_state);
@@ -1080,14 +1073,6 @@
return encoder;
}
-scoped_ptr<media::VideoEncodeAccelerator> VEAClient::CreateVTVEA() {
- scoped_ptr<media::VideoEncodeAccelerator> encoder;
-#if defined(OS_MACOSX)
- encoder.reset(new VTVideoEncodeAccelerator());
-#endif
- return encoder;
-}
-
void VEAClient::CreateEncoder() {
DCHECK(thread_checker_.CalledOnValidThread());
LOG_ASSERT(!has_encoder());
@@ -1095,8 +1080,7 @@
scoped_ptr<media::VideoEncodeAccelerator> encoders[] = {
CreateFakeVEA(),
CreateV4L2VEA(),
- CreateVaapiVEA(),
- CreateVTVEA()
+ CreateVaapiVEA()
};
DVLOG(1) << "Profile: " << test_stream_->requested_profile
@@ -1667,7 +1651,6 @@
encoder_thread.Stop();
}
-#if !defined(OS_MACOSX)
INSTANTIATE_TEST_CASE_P(
SimpleEncode,
VideoEncodeAcceleratorTest,
@@ -1712,26 +1695,6 @@
base::MakeTuple(3, false, 0, false, false, false, false, false),
base::MakeTuple(3, false, 0, true, false, false, true, false),
base::MakeTuple(3, false, 0, true, false, true, false, false)));
-#else
-INSTANTIATE_TEST_CASE_P(
- SimpleEncode,
- VideoEncodeAcceleratorTest,
- ::testing::Values(
- base::MakeTuple(1, true, 0, false, false, false, false, false),
- base::MakeTuple(1, true, 0, false, false, false, false, true)));
-
-INSTANTIATE_TEST_CASE_P(
- EncoderPerf,
- VideoEncodeAcceleratorTest,
- ::testing::Values(
- base::MakeTuple(1, false, 0, false, true, false, false, false)));
-
-INSTANTIATE_TEST_CASE_P(
- MultipleEncoders,
- VideoEncodeAcceleratorTest,
- ::testing::Values(
- base::MakeTuple(3, false, 0, false, false, false, false, false)));
-#endif
// TODO(posciak): more tests:
// - async FeedEncoderWithOutput
« no previous file with comments | « content/common/gpu/media/gpu_video_encode_accelerator.cc ('k') | content/common/gpu/media/vt_video_encode_accelerator_mac.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698