From a4dcb749fbbc83a874d4e2c65de5a98465d3e200 Mon Sep 17 00:00:00 2001 From: Elad Alon Date: Fri, 30 Nov 2018 15:18:14 +0100 Subject: [PATCH] Fix output period in RtcEventLogImpl MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit RtcEventLogImpl::StartLogging() was ignoring one of its parameters. This CL fixes the issue. Bug: webrtc:10082 Change-Id: Ie1790c1a7299748dabe99909d967384ad9895635 Reviewed-on: https://webrtc-review.googlesource.com/c/112586 Reviewed-by: Björn Terelius Commit-Queue: Elad Alon Cr-Commit-Position: refs/heads/master@{#25858} --- logging/rtc_event_log/rtc_event_log_impl.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/logging/rtc_event_log/rtc_event_log_impl.cc b/logging/rtc_event_log/rtc_event_log_impl.cc index c022a3d418..441f22267a 100644 --- a/logging/rtc_event_log/rtc_event_log_impl.cc +++ b/logging/rtc_event_log/rtc_event_log_impl.cc @@ -179,10 +179,11 @@ bool RtcEventLogImpl::StartLogging(std::unique_ptr output, << "(" << timestamp_us << ", " << utc_time_us << ")."; // Binding to |this| is safe because |this| outlives the |task_queue_|. - auto start = [this, timestamp_us, + auto start = [this, output_period_ms, timestamp_us, utc_time_us](std::unique_ptr output) { RTC_DCHECK_RUN_ON(task_queue_.get()); RTC_DCHECK(output->IsActive()); + output_period_ms_ = output_period_ms; event_output_ = std::move(output); num_config_events_written_ = 0; WriteToOutput(event_encoder_->EncodeLogStart(timestamp_us, utc_time_us));