Speculative Revert: "Use FakeRenderer when fuzzing"

This reverts commit ce9da1636aba347f452f33a00a75b929eee77570.
The vp8_replay_fuzzer runs out of memory unders MSAN for the input
in bug 1015797.

Tbr: kcwu@chromium.org
Bug: chromium:1015797, chromium:952606, chromium:1009077, chromium:1009073
Change-Id: Iab03437595b33e56816efe83b74fab6faf2340da
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/158402
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#29609}
This commit is contained in:
Patrik Höglund
2019-10-25 10:23:34 +02:00
committed by Commit Bot
parent b394a565ab
commit 4f2783b9fe
3 changed files with 5 additions and 3 deletions

View File

@ -35,7 +35,7 @@ rtc_library("rtp_replayer") {
"../../../test:rtp_test_utils",
"../../../test:run_test",
"../../../test:run_test_interface",
"../../../test:test_common",
"../../../test:test_renderer",
"../../../test:test_support",
"../../../test:video_test_common",
]

View File

@ -21,7 +21,6 @@
#include "test/call_config_utils.h"
#include "test/encoder_settings.h"
#include "test/fake_decoder.h"
#include "test/fake_videorenderer.h"
#include "test/rtp_file_reader.h"
#include "test/rtp_header_parser.h"
@ -112,7 +111,9 @@ void RtpReplayer::SetupVideoStreams(
decoder.decoder_factory = stream_state->decoder_factory.get();
}
stream_state->sinks.emplace_back(new test::FakeVideoRenderer());
// Create the window to display the rendered video.
stream_state->sinks.emplace_back(
test::VideoRenderer::Create("Fuzzing WebRTC Video Config", 640, 480));
// Create a receive stream for this config.
receive_config.renderer = stream_state->sinks.back().get();
stream_state->receive_streams.emplace_back(

View File

@ -28,6 +28,7 @@
#include "test/null_transport.h"
#include "test/rtp_file_reader.h"
#include "test/test_video_capturer.h"
#include "test/video_renderer.h"
namespace webrtc {
namespace test {