Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(320)

Unified Diff: webrtc/logging/rtc_event_log/rtc_event_log_unittest_helper.cc

Issue 2808833002: Delay based logging. (Closed)
Patch Set: Unittest fix. Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/logging/rtc_event_log/rtc_event_log_parser.cc ('k') | webrtc/tools/event_log_visualizer/analyzer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/logging/rtc_event_log/rtc_event_log_unittest_helper.cc
diff --git a/webrtc/logging/rtc_event_log/rtc_event_log_unittest_helper.cc b/webrtc/logging/rtc_event_log/rtc_event_log_unittest_helper.cc
index 7519ee5933209a444ce7c08acc4c6d289dd1e321..6cbb23e59f7a08a6a85cdf5e45ca0acdaef399f5 100644
--- a/webrtc/logging/rtc_event_log/rtc_event_log_unittest_helper.cc
+++ b/webrtc/logging/rtc_event_log/rtc_event_log_unittest_helper.cc
@@ -538,12 +538,10 @@ void RtcEventLogTestHelper::VerifyBweDelayEvent(
GetRuntimeDetectorState(bwe_event.detector_state()));
// Check consistency of the parser.
- int32_t parsed_bitrate;
- BandwidthUsage parsed_detector_state;
- parsed_log.GetDelayBasedBweUpdate(index, &parsed_bitrate,
- &parsed_detector_state);
- EXPECT_EQ(bitrate, parsed_bitrate);
- EXPECT_EQ(detector_state, parsed_detector_state);
+ ParsedRtcEventLog::BweDelayBasedUpdate res =
+ parsed_log.GetDelayBasedBweUpdate(index);
+ EXPECT_EQ(res.bitrate_bps, bitrate);
+ EXPECT_EQ(res.detector_state, detector_state);
}
void RtcEventLogTestHelper::VerifyAudioNetworkAdaptation(
« no previous file with comments | « webrtc/logging/rtc_event_log/rtc_event_log_parser.cc ('k') | webrtc/tools/event_log_visualizer/analyzer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698