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

Unified Diff: webrtc/audio/test/low_bandwidth_audio_test.h

Issue 2931873002: Test and fix for huge bwe drop after alr state. (Closed)
Patch Set: Respons to comments Created 3 years, 6 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/audio/BUILD.gn ('k') | webrtc/audio/test/low_bandwidth_audio_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/audio/test/low_bandwidth_audio_test.h
diff --git a/webrtc/audio/test/low_bandwidth_audio_test.h b/webrtc/audio/test/low_bandwidth_audio_test.h
index eae650af22ca167fde67aff7713f3391d1e554cf..0e04baa991bb85c08ae3863253f60512ebd1ce8b 100644
--- a/webrtc/audio/test/low_bandwidth_audio_test.h
+++ b/webrtc/audio/test/low_bandwidth_audio_test.h
@@ -51,6 +51,8 @@ class AudioQualityTest : public test::EndToEndTest {
void PerformTest() override;
void OnTestFinished() override;
+ Call* sender_call_;
holmer 2017/06/19 08:53:49 AFAICT this can still be private?
tschumi 2017/06/19 10:36:03 I think i could move it in to the NoBandwidthDropA
holmer 2017/06/19 11:51:53 Ok, maybe worth doing? Or protected is enough?
tschumi 2017/06/19 12:08:48 Ok moved sender_call_ to NoBandwidthDropAfterDtx.
+
private:
test::FakeAudioDevice* send_audio_device_;
};
« no previous file with comments | « webrtc/audio/BUILD.gn ('k') | webrtc/audio/test/low_bandwidth_audio_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698