From bb66ec35739830847bfb0146cd029ca41421b2d8 Mon Sep 17 00:00:00 2001 From: skvlad Date: Wed, 14 Dec 2016 01:17:28 -0800 Subject: [PATCH] Disable flaky test VideoProcessorIntegrationTest.ProcessNoLossChangeFrameRateFrameDropVP9 This test is flaky on all platforms, not just Android. Disabling it entirely until webrtc:6057 is fixed. BUG=webrtc:6057 Review-Url: https://codereview.webrtc.org/2568743007 Cr-Commit-Position: refs/heads/master@{#15594} --- .../codecs/test/videoprocessor_integrationtest.cc | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc b/webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc index ee8f355085..5a202a1e3c 100644 --- a/webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc +++ b/webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc @@ -703,16 +703,10 @@ TEST_F(VideoProcessorIntegrationTest, ProcessNoLossChangeBitRateVP9) { // Note: quality after update should be higher but we currently compute quality // metrics averaged over whole sequence run. -#if defined(WEBRTC_ANDROID) -// Flaky on Android: https://bugs.chromium.org/p/webrtc/issues/detail?id=6057. -#define MAYBE_ProcessNoLossChangeFrameRateFrameDropVP9 \ - DISABLED_ProcessNoLossChangeFrameRateFrameDropVP9 -#else -#define MAYBE_ProcessNoLossChangeFrameRateFrameDropVP9 \ - ProcessNoLossChangeFrameRateFrameDropVP9 -#endif + +// Flaky https://bugs.chromium.org/p/webrtc/issues/detail?id=6057. TEST_F(VideoProcessorIntegrationTest, - MAYBE_ProcessNoLossChangeFrameRateFrameDropVP9) { + DISABLED_ProcessNoLossChangeFrameRateFrameDropVP9) { config_.networking_config.packet_loss_probability = 0; // Bitrate and frame rate profile. RateProfile rate_profile;