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

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: Implementing onFirstMediaPacketReceived in objc AppRTC client. Created 4 years, 11 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 983 matching lines...) Expand 10 before | Expand all | Expand 10 after
994 public void onDataChannel(final DataChannel dc) { 994 public void onDataChannel(final DataChannel dc) {
995 reportError("AppRTC doesn't use data channels, but got: " + dc.label() 995 reportError("AppRTC doesn't use data channels, but got: " + dc.label()
996 + " anyway!"); 996 + " anyway!");
997 } 997 }
998 998
999 @Override 999 @Override
1000 public void onRenegotiationNeeded() { 1000 public void onRenegotiationNeeded() {
1001 // No need to do anything; AppRTC follows a pre-agreed-upon 1001 // No need to do anything; AppRTC follows a pre-agreed-upon
1002 // signaling/negotiation protocol. 1002 // signaling/negotiation protocol.
1003 } 1003 }
1004
1005 @Override
1006 public void onFirstMediaPacketReceived() {
1007 }
1004 } 1008 }
1005 1009
1006 // Implementation detail: handle offer creation/signaling and answer setting, 1010 // Implementation detail: handle offer creation/signaling and answer setting,
1007 // as well as adding remote ICE candidates once the answer SDP is set. 1011 // as well as adding remote ICE candidates once the answer SDP is set.
1008 private class SDPObserver implements SdpObserver { 1012 private class SDPObserver implements SdpObserver {
1009 @Override 1013 @Override
1010 public void onCreateSuccess(final SessionDescription origSdp) { 1014 public void onCreateSuccess(final SessionDescription origSdp) {
1011 if (localSdp != null) { 1015 if (localSdp != null) {
1012 reportError("Multiple SDP create."); 1016 reportError("Multiple SDP create.");
1013 return; 1017 return;
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
1077 public void onCreateFailure(final String error) { 1081 public void onCreateFailure(final String error) {
1078 reportError("createSDP error: " + error); 1082 reportError("createSDP error: " + error);
1079 } 1083 }
1080 1084
1081 @Override 1085 @Override
1082 public void onSetFailure(final String error) { 1086 public void onSetFailure(final String error) {
1083 reportError("setSDP error: " + error); 1087 reportError("setSDP error: " + error);
1084 } 1088 }
1085 } 1089 }
1086 } 1090 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698