Commit 35daddd7 authored by David Staessens's avatar David Staessens Committed by Commit Bot

media/gpu/test: Fix member var not being initialized in video decoder tests.

This CL fixes a member variable not being properly initialized in the
video decoder tests before first using it. In practice this shouldn't
have caused any bugs.

TEST=./video_decode_accelerator_tests on hana

BUG=None

Change-Id: I9ff4cf8e8415322a78ccd66d699e86a47e191f6d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2087854Reviewed-by: default avatarHirokazu Honda <hiroh@chromium.org>
Commit-Queue: David Staessens <dstaessens@chromium.org>
Cr-Commit-Position: refs/heads/master@{#747139}
parent 3d0e9836
...@@ -129,7 +129,7 @@ void VideoPlayer::Play() { ...@@ -129,7 +129,7 @@ void VideoPlayer::Play() {
DVLOGF(4); DVLOGF(4);
// Play until the end of the video. // Play until the end of the video.
PlayUntil(VideoPlayerEvent::kNumEvents, 0); PlayUntil(VideoPlayerEvent::kNumEvents, std::numeric_limits<size_t>::max());
} }
void VideoPlayer::PlayUntil(VideoPlayerEvent event, size_t event_count) { void VideoPlayer::PlayUntil(VideoPlayerEvent event, size_t event_count) {
......
...@@ -5,6 +5,7 @@ ...@@ -5,6 +5,7 @@
#ifndef MEDIA_GPU_TEST_VIDEO_PLAYER_VIDEO_PLAYER_H_ #ifndef MEDIA_GPU_TEST_VIDEO_PLAYER_VIDEO_PLAYER_H_
#define MEDIA_GPU_TEST_VIDEO_PLAYER_VIDEO_PLAYER_H_ #define MEDIA_GPU_TEST_VIDEO_PLAYER_VIDEO_PLAYER_H_
#include <limits.h>
#include <memory> #include <memory>
#include <utility> #include <utility>
#include <vector> #include <vector>
...@@ -155,7 +156,8 @@ class VideoPlayer { ...@@ -155,7 +156,8 @@ class VideoPlayer {
// Automatically pause decoding once the video player has seen the specified // Automatically pause decoding once the video player has seen the specified
// number of events occur. // number of events occur.
std::pair<VideoPlayerEvent, size_t> play_until_; std::pair<VideoPlayerEvent, size_t> play_until_{
VideoPlayerEvent::kNumEvents, std::numeric_limits<size_t>::max()};
SEQUENCE_CHECKER(sequence_checker_); SEQUENCE_CHECKER(sequence_checker_);
DISALLOW_COPY_AND_ASSIGN(VideoPlayer); DISALLOW_COPY_AND_ASSIGN(VideoPlayer);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment