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

Unified Diff: webrtc/modules/audio_coding/audio_network_adaptor/event_log_writer.cc

Issue 3016473002: Remove encoding code from RtcEventLogImpl and use RtcEventLogEncoder instead (Closed)
Patch Set: Created 3 years, 3 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
Index: webrtc/modules/audio_coding/audio_network_adaptor/event_log_writer.cc
diff --git a/webrtc/modules/audio_coding/audio_network_adaptor/event_log_writer.cc b/webrtc/modules/audio_coding/audio_network_adaptor/event_log_writer.cc
index b4fcbfd5ead1cd92eecc99d734e997276b422780..7480d61491bb5bf84141e4cdc343593731783ce3 100644
--- a/webrtc/modules/audio_coding/audio_network_adaptor/event_log_writer.cc
+++ b/webrtc/modules/audio_coding/audio_network_adaptor/event_log_writer.cc
@@ -9,10 +9,13 @@
*/
#include <math.h>
+
#include <algorithm>
+#include "webrtc/logging/rtc_event_log/events/rtc_event_audio_network_adaptation.h"
#include "webrtc/logging/rtc_event_log/rtc_event_log.h"
#include "webrtc/modules/audio_coding/audio_network_adaptor/event_log_writer.h"
+#include "webrtc/rtc_base/ptr_util.h"
namespace webrtc {
@@ -60,7 +63,9 @@ void EventLogWriter::MaybeLogEncoderConfig(
}
void EventLogWriter::LogEncoderConfig(const AudioEncoderRuntimeConfig& config) {
- event_log_->LogAudioNetworkAdaptation(config);
+ auto config_copy = rtc::MakeUnique<AudioEncoderRuntimeConfig>(config);
+ event_log_->Log(
+ rtc::MakeUnique<RtcEventAudioNetworkAdaptation>(std::move(config_copy)));
last_logged_config_ = config;
}

Powered by Google App Engine
This is Rietveld 408576698