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

Unified Diff: webrtc/examples/androidtests/src/org/appspot/apprtc/test/PeerConnectionClientTest.java

Issue 2002093002: Add an option to disable built-in AEC to AppRTC Android Demo (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Remove empty line Created 4 years, 7 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/examples/androidtests/src/org/appspot/apprtc/test/PeerConnectionClientTest.java
diff --git a/webrtc/examples/androidtests/src/org/appspot/apprtc/test/PeerConnectionClientTest.java b/webrtc/examples/androidtests/src/org/appspot/apprtc/test/PeerConnectionClientTest.java
index bba182fa1ec1d80b2c9d810bc1fea44213bda890..1f0b60638566b465f218f3eceff519faefc76f66 100644
--- a/webrtc/examples/androidtests/src/org/appspot/apprtc/test/PeerConnectionClientTest.java
+++ b/webrtc/examples/androidtests/src/org/appspot/apprtc/test/PeerConnectionClientTest.java
@@ -263,9 +263,24 @@ public class PeerConnectionClientTest extends InstrumentationTestCase
private PeerConnectionParameters createParametersForAudioCall() {
PeerConnectionParameters peerConnectionParameters =
new PeerConnectionParameters(
- false, true, false, // videoCallEnabled, loopback, tracing.
- 0, 0, 0, 0, "", true, false, // video codec parameters.
- 0, "OPUS", false, false, false); // audio codec parameters.
+ false, /* videoCallEnabled */
+ true, /* loopback */
+ false, /* tracing */
+ // Video codec parameters.
+ 0, /* videoWidth */
+ 0, /* videoHeight */
+ 0, /* videoFps */
+ 0, /* videoStartBitrate */
+ "", /* videoCodec */
+ true, /* videoCodecHwAcceleration */
+ false, /* captureToToTexture */
+ // Audio codec parameters.
+ 0, /* audioStartBitrate */
+ "OPUS", /* audioCodec */
+ false, /* noAudioProcessing */
+ false, /* aecDump */
+ false /* useOpenSLES */,
+ false /* disableBuiltInAEC */);
return peerConnectionParameters;
}
@@ -273,12 +288,28 @@ public class PeerConnectionClientTest extends InstrumentationTestCase
String videoCodec, boolean captureToTexture) {
PeerConnectionParameters peerConnectionParameters =
new PeerConnectionParameters(
- true, true, false, // videoCallEnabled, loopback, tracing.
- 0, 0, 0, 0, videoCodec, true, captureToTexture, // video codec parameters.
- 0, "OPUS", false, false, false); // audio codec parameters.
+ true, /* videoCallEnabled */
+ true, /* loopback */
+ false, /* tracing */
+ // Video codec parameters.
+ 0, /* videoWidth */
+ 0, /* videoHeight */
+ 0, /* videoFps */
+ 0, /* videoStartBitrate */
+ videoCodec, /* videoCodec */
+ true, /* videoCodecHwAcceleration */
+ captureToTexture, /* captureToToTexture */
+ // Audio codec parameters.
+ 0, /* audioStartBitrate */
+ "OPUS", /* audioCodec */
+ false, /* noAudioProcessing */
+ false, /* aecDump */
+ false /* useOpenSLES */,
+ false /* disableBuiltInAEC */);
return peerConnectionParameters;
}
+
@Override
public void setUp() {
signalingExecutor = Executors.newSingleThreadExecutor();

Powered by Google App Engine
This is Rietveld 408576698