Change NetEq::GetPlayoutTimestamp to return an rtc::Optional<uint32_t>
This is in preparation for changes to when the playout timestamp is
valid.
BUG=webrtc:5669
Review URL: https://codereview.webrtc.org/1853183002
Cr-Commit-Position: refs/heads/master@{#12256}
diff --git a/webrtc/modules/audio_coding/neteq/neteq_unittest.cc b/webrtc/modules/audio_coding/neteq/neteq_unittest.cc
index 340cf58..00a878c 100644
--- a/webrtc/modules/audio_coding/neteq/neteq_unittest.cc
+++ b/webrtc/modules/audio_coding/neteq/neteq_unittest.cc
@@ -1522,9 +1522,9 @@
}
uint32_t NetEqDecodingTest::PlayoutTimestamp() {
- uint32_t playout_timestamp = 0;
- EXPECT_TRUE(neteq_->GetPlayoutTimestamp(&playout_timestamp));
- return playout_timestamp;
+ rtc::Optional<uint32_t> playout_timestamp = neteq_->GetPlayoutTimestamp();
+ EXPECT_TRUE(playout_timestamp);
+ return playout_timestamp.value_or(0);
}
TEST_F(NetEqDecodingTest, DiscardDuplicateCng) { DuplicateCng(); }
@@ -1570,5 +1570,4 @@
// Verify speech output.
EXPECT_EQ(AudioFrame::kNormalSpeech, out_frame_.speech_type_);
}
-
} // namespace webrtc