Fixed flaky VideoSendStreamTest::SupportsAbsoluteSendTime

webrtc/video/video_send_stream_tests.cc

Index: webrtc/video/video_send_stream_tests.cc
diff --git a/webrtc/video/video_send_stream_tests.cc b/webrtc/video/video_send_stream_tests.cc
index 623cd34ac5e8860f7b9e6666172f5a082134ffa7..5dbfa597def9ed887e49c39501712230322b920c 100644
--- a/webrtc/video/video_send_stream_tests.cc
+++ b/webrtc/video/video_send_stream_tests.cc
@@ -136,7 +136,6 @@ TEST_F(VideoSendStreamTest, SupportsAbsoluteSendTime) {
       EXPECT_FALSE(header.extension.hasTransmissionTimeOffset);
       EXPECT_TRUE(header.extension.hasAbsoluteSendTime);
       EXPECT_EQ(header.extension.transmissionTimeOffset, 0);
-      EXPECT_GT(header.extension.absoluteSendTime, 0u);

[pbos-webrtc, 2016/09/01 23:06:22]

How about returning SEND_PACKET if absoluteSendTime is non-zero and wait for the
next one to arrive instead (and log a warning, so that you can see why (through
repeated logging) if the test times out)?

[pbos-webrtc, 2016/09/01 23:06:49]

Sorry, return SEND_PACKET early if it's zero, not non-zero.

[skvlad, 2016/09/01 23:39:28]

Done.

       observation_complete_.Set();
 
       return SEND_PACKET;