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

Unified Diff: webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.h

Issue 2362003002: Revert of Adding debug dump to audio network adaptor. (Closed)
Patch Set: Created 4 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/audio_network_adaptor_impl.h
diff --git a/webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.h b/webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.h
index 289b6777e5313a33422ddb6b29c0d75c4af18b79..6f8d348d0373b820813b2d354b8ce1e77d66278f 100644
--- a/webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.h
+++ b/webrtc/modules/audio_coding/audio_network_adaptor/audio_network_adaptor_impl.h
@@ -16,9 +16,7 @@
#include "webrtc/base/constructormagic.h"
#include "webrtc/modules/audio_coding/audio_network_adaptor/controller.h"
#include "webrtc/modules/audio_coding/audio_network_adaptor/controller_manager.h"
-#include "webrtc/modules/audio_coding/audio_network_adaptor/debug_dump_writer.h"
#include "webrtc/modules/audio_coding/audio_network_adaptor/include/audio_network_adaptor.h"
-#include "webrtc/system_wrappers/include/clock.h"
namespace webrtc {
@@ -27,26 +25,17 @@
struct Config {
Config();
~Config();
- const Clock* clock;
};
AudioNetworkAdaptorImpl(
const Config& config,
std::unique_ptr<ControllerManager> controller_manager);
- // Dependency injection for testing.
- AudioNetworkAdaptorImpl(
- const Config& config,
- std::unique_ptr<ControllerManager> controller_manager,
- std::unique_ptr<DebugDumpWriter> debug_dump_writer = nullptr);
-
~AudioNetworkAdaptorImpl() override;
void SetUplinkBandwidth(int uplink_bandwidth_bps) override;
void SetUplinkPacketLossFraction(float uplink_packet_loss_fraction) override;
-
- void SetRtt(int rtt_ms) override;
void SetReceiverFrameLengthRange(int min_frame_length_ms,
int max_frame_length_ms) override;
@@ -55,16 +44,10 @@
void StartDebugDump(FILE* file_handle) override;
- void StopDebugDump() override;
-
private:
- void DumpNetworkMetrics();
-
const Config config_;
std::unique_ptr<ControllerManager> controller_manager_;
-
- std::unique_ptr<DebugDumpWriter> debug_dump_writer_;
Controller::NetworkMetrics last_metrics_;

Powered by Google App Engine
This is Rietveld 408576698