From 03c592a1e9f9dbad02bfc9d1f55d8b8c5c499208 Mon Sep 17 00:00:00 2001 From: Artem Titarenko Date: Mon, 24 Sep 2018 11:22:17 +0200 Subject: [PATCH] Disabled TestPacketBuffer.SeqNumWrapOneFrame test due to clang update MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Until further investigation. Clang update: chromium:880827 Bug: chromium:887464 Change-Id: Id1fe85a013920e6ae8c6ac69efb0a0502b9dd6fe Reviewed-on: https://webrtc-review.googlesource.com/101561 Commit-Queue: Artem Titarenko Reviewed-by: Artem Titarenko Reviewed-by: Patrik Höglund Cr-Commit-Position: refs/heads/master@{#24795} --- modules/video_coding/video_packet_buffer_unittest.cc | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/modules/video_coding/video_packet_buffer_unittest.cc b/modules/video_coding/video_packet_buffer_unittest.cc index 50f5a5c492..8866368c3b 100644 --- a/modules/video_coding/video_packet_buffer_unittest.cc +++ b/modules/video_coding/video_packet_buffer_unittest.cc @@ -112,7 +112,14 @@ TEST_F(TestPacketBuffer, InsertDuplicatePacket) { EXPECT_TRUE(Insert(seq_num, kKeyFrame, kFirst, kLast)); } -TEST_F(TestPacketBuffer, SeqNumWrapOneFrame) { +#if defined(WEBRTC_ANDROID) +// Fails on android after clang update +// TODO(crbug.com/887464): Reenable this +#define MAYBE_SeqNumWrapOneFrame DISABLED_SeqNumWrapOneFrame +#else +#define MAYBE_SeqNumWrapOneFrame SeqNumWrapOneFrame +#endif +TEST_F(TestPacketBuffer, MAYBE_SeqNumWrapOneFrame) { EXPECT_TRUE(Insert(0xFFFF, kKeyFrame, kFirst, kNotLast)); EXPECT_TRUE(Insert(0x0, kKeyFrame, kNotFirst, kLast));