zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Disabled TestPacketBuffer.SeqNumWrapOneFrame test due to clang update

Browse Source 
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 <artit@webrtc.org>
Reviewed-by: Artem Titarenko <artit@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24795}
This commit is contained in:
Artem Titarenko
2018-09-24 11:22:17 +02:00
committed by Commit Bot
parent 74e3742635
commit 03c592a1e9
1 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
modules/video_coding/video_packet_buffer_unittest.cc
View File

@ -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));