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

Log current time if we stop logging before stop_time_.

Browse Source 
This happens if we stop logging because we have reached the file size limit. The large timestamp causes problems in the analysis tool.

BUG=webrtc:6138

Review-Url: https://codereview.webrtc.org/2175713002
Cr-Commit-Position: refs/heads/master@{#13581}
This commit is contained in:
terelius
2016-07-29 12:38:39 -07:00
committed by Commit bot
parent 7a11ae3c7c
commit 76a44d5daf
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
webrtc/call/rtc_event_log_helper_thread.cc
View File

@ -224,7 +224,11 @@ void RtcEventLogHelperThread::StopLogFile() {
output_string_.clear();
rtclog::Event end_event;
end_event.set_timestamp_us(stop_time_);
// This function can be called either because we have reached the stop time,
// or because we have reached the log file size limit. Therefore, use the
// current time if we have not reached the time limit.
end_event.set_timestamp_us(
std::min(stop_time_, clock_->TimeInMicroseconds()));
end_event.set_type(rtclog::Event::LOG_END);
AppendEventToString(&end_event);