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

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

Issue 1648813004: Remove candidates when doing continual gathering (Closed) Base URL: https://chromium.googlesource.com/external/webrtc@master
Patch Set: small fixes Created 4 years, 9 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright 2014 The WebRTC Project Authors. All rights reserved. 2 * Copyright 2014 The WebRTC Project Authors. All rights reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 125 matching lines...) Expand 10 before | Expand all | Expand 10 after
136 pcClient.addRemoteIceCandidate(candidate); 136 pcClient.addRemoteIceCandidate(candidate);
137 } 137 }
138 }); 138 });
139 } 139 }
140 iceCandidates.add(candidate); 140 iceCandidates.add(candidate);
141 iceCandidateEvent.notifyAll(); 141 iceCandidateEvent.notifyAll();
142 } 142 }
143 } 143 }
144 144
145 @Override 145 @Override
146 public void onIceCandidatesRemoved(final IceCandidate[] candidates) {
147 // TODO(honghaiz): Add this for tests.
148 }
149
150 @Override
146 public void onIceConnected() { 151 public void onIceConnected() {
147 Log.d(TAG, "ICE Connected"); 152 Log.d(TAG, "ICE Connected");
148 synchronized(iceConnectedEvent) { 153 synchronized(iceConnectedEvent) {
149 isIceConnected = true; 154 isIceConnected = true;
150 iceConnectedEvent.notifyAll(); 155 iceConnectedEvent.notifyAll();
151 } 156 }
152 } 157 }
153 158
154 @Override 159 @Override
155 public void onIceDisconnected() { 160 public void onIceDisconnected() {
(...skipping 389 matching lines...) Expand 10 before | Expand all | Expand 10 after
545 assertTrue("Local video frames were not rendered after video restart.", 550 assertTrue("Local video frames were not rendered after video restart.",
546 localRenderer.waitForFramesRendered(WAIT_TIMEOUT)); 551 localRenderer.waitForFramesRendered(WAIT_TIMEOUT));
547 assertTrue("Remote video frames were not rendered after video restart.", 552 assertTrue("Remote video frames were not rendered after video restart.",
548 remoteRenderer.waitForFramesRendered(WAIT_TIMEOUT)); 553 remoteRenderer.waitForFramesRendered(WAIT_TIMEOUT));
549 } 554 }
550 pcClient.close(); 555 pcClient.close();
551 assertTrue(waitForPeerConnectionClosed(WAIT_TIMEOUT)); 556 assertTrue(waitForPeerConnectionClosed(WAIT_TIMEOUT));
552 Log.d(TAG, "testVideoSourceRestart done."); 557 Log.d(TAG, "testVideoSourceRestart done.");
553 } 558 }
554 } 559 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698