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

Side by Side Diff: webrtc/examples/androidapp/src/org/appspot/apprtc/PeerConnectionClient.java

Issue 1581693006: Adding "first packet received" notification to PeerConnectionObserver. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Fixing race condition in unit test. Created 4 years, 10 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
« no previous file with comments | « talk/app/webrtc/webrtcsession.cc ('k') | webrtc/examples/objc/AppRTCDemo/ARDAppClient.m » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 982 matching lines...) Expand 10 before | Expand all | Expand 10 after
993 public void onDataChannel(final DataChannel dc) { 993 public void onDataChannel(final DataChannel dc) {
994 reportError("AppRTC doesn't use data channels, but got: " + dc.label() 994 reportError("AppRTC doesn't use data channels, but got: " + dc.label()
995 + " anyway!"); 995 + " anyway!");
996 } 996 }
997 997
998 @Override 998 @Override
999 public void onRenegotiationNeeded() { 999 public void onRenegotiationNeeded() {
1000 // No need to do anything; AppRTC follows a pre-agreed-upon 1000 // No need to do anything; AppRTC follows a pre-agreed-upon
1001 // signaling/negotiation protocol. 1001 // signaling/negotiation protocol.
1002 } 1002 }
1003
1004 @Override
1005 public void onFirstMediaPacketReceived() {
1006 }
1003 } 1007 }
1004 1008
1005 // Implementation detail: handle offer creation/signaling and answer setting, 1009 // Implementation detail: handle offer creation/signaling and answer setting,
1006 // as well as adding remote ICE candidates once the answer SDP is set. 1010 // as well as adding remote ICE candidates once the answer SDP is set.
1007 private class SDPObserver implements SdpObserver { 1011 private class SDPObserver implements SdpObserver {
1008 @Override 1012 @Override
1009 public void onCreateSuccess(final SessionDescription origSdp) { 1013 public void onCreateSuccess(final SessionDescription origSdp) {
1010 if (localSdp != null) { 1014 if (localSdp != null) {
1011 reportError("Multiple SDP create."); 1015 reportError("Multiple SDP create.");
1012 return; 1016 return;
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
1076 public void onCreateFailure(final String error) { 1080 public void onCreateFailure(final String error) {
1077 reportError("createSDP error: " + error); 1081 reportError("createSDP error: " + error);
1078 } 1082 }
1079 1083
1080 @Override 1084 @Override
1081 public void onSetFailure(final String error) { 1085 public void onSetFailure(final String error) {
1082 reportError("setSDP error: " + error); 1086 reportError("setSDP error: " + error);
1083 } 1087 }
1084 } 1088 }
1085 } 1089 }
OLDNEW
« no previous file with comments | « talk/app/webrtc/webrtcsession.cc ('k') | webrtc/examples/objc/AppRTCDemo/ARDAppClient.m » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698