Skip to content

Commit

Permalink
media/gpu/test: Add multiple concurrent encodes test to new video enc…
Browse files Browse the repository at this point in the history
…oder tests.

This CL adds a new test to the video_encode_accelerator_tests that
tests encoding multiple videos simultaneously from start to finish.

Bug: 1045825
Test: ./video_encode_accelerator_tests on hana
Change-Id: I6283530ad3a2b0c5f8c9bb62ce474dad07748a7b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2228012
Commit-Queue: David Staessens <dstaessens@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#778156}
  • Loading branch information
David Staessens authored and Commit Bot committed Jun 15, 2020
1 parent 0c8806f commit 6d5d687
Showing 1 changed file with 25 additions and 0 deletions.
25 changes: 25 additions & 0 deletions media/gpu/video_encode_accelerator_tests.cc
Original file line number Diff line number Diff line change
Expand Up @@ -192,6 +192,31 @@ TEST_F(VideoEncoderTest, DestroyBeforeInitialize) {
EXPECT_NE(video_encoder, nullptr);
}

// Encode multiple videos simultaneously from start to finish.
TEST_F(VideoEncoderTest, FlushAtEndOfStream_MultipleConcurrentEncodes) {
// The minimal number of concurrent encoders we expect to be supported.
constexpr size_t kMinSupportedConcurrentEncoders = 3;

VideoEncoderClientConfig config = VideoEncoderClientConfig();
config.framerate = g_env->Video()->FrameRate();

std::vector<std::unique_ptr<VideoEncoder>> encoders(
kMinSupportedConcurrentEncoders);
for (size_t i = 0; i < kMinSupportedConcurrentEncoders; ++i)
encoders[i] = CreateVideoEncoder(g_env->Video(), config);

for (size_t i = 0; i < kMinSupportedConcurrentEncoders; ++i)
encoders[i]->Encode();

for (size_t i = 0; i < kMinSupportedConcurrentEncoders; ++i) {
EXPECT_TRUE(encoders[i]->WaitForFlushDone());
EXPECT_EQ(encoders[i]->GetFlushDoneCount(), 1u);
EXPECT_EQ(encoders[i]->GetFrameReleasedCount(),
g_env->Video()->NumFrames());
EXPECT_TRUE(encoders[i]->WaitForBitstreamProcessors());
}
}

} // namespace test
} // namespace media

Expand Down

0 comments on commit 6d5d687

Please sign in to comment.