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

Side by Side Diff: talk/app/webrtc/peerconnection.h

Issue 1640173004: Revert of Adding "first packet received" notification to PeerConnectionObserver. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: 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
OLDNEW
1 /* 1 /*
2 * libjingle 2 * libjingle
3 * Copyright 2012 Google Inc. 3 * Copyright 2012 Google Inc.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions are met: 6 * modification, are permitted provided that the following conditions are met:
7 * 7 *
8 * 1. Redistributions of source code must retain the above copyright notice, 8 * 1. Redistributions of source code must retain the above copyright notice,
9 * this list of conditions and the following disclaimer. 9 * this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright notice, 10 * 2. Redistributions in binary form must reproduce the above copyright notice,
(...skipping 303 matching lines...) Expand 10 before | Expand all | Expand 10 after
314 314
315 // Notifications from WebRtcSession relating to BaseChannels. 315 // Notifications from WebRtcSession relating to BaseChannels.
316 void OnVoiceChannelDestroyed(); 316 void OnVoiceChannelDestroyed();
317 void OnVideoChannelDestroyed(); 317 void OnVideoChannelDestroyed();
318 void OnDataChannelCreated(); 318 void OnDataChannelCreated();
319 void OnDataChannelDestroyed(); 319 void OnDataChannelDestroyed();
320 // Called when the cricket::DataChannel receives a message indicating that a 320 // Called when the cricket::DataChannel receives a message indicating that a
321 // webrtc::DataChannel should be opened. 321 // webrtc::DataChannel should be opened.
322 void OnDataChannelOpenMessage(const std::string& label, 322 void OnDataChannelOpenMessage(const std::string& label,
323 const InternalDataChannelInit& config); 323 const InternalDataChannelInit& config);
324 void OnFirstMediaPacketReceived();
325 324
326 RtpSenderInterface* FindSenderById(const std::string& id); 325 RtpSenderInterface* FindSenderById(const std::string& id);
327 326
328 std::vector<rtc::scoped_refptr<RtpSenderInterface>>::iterator 327 std::vector<rtc::scoped_refptr<RtpSenderInterface>>::iterator
329 FindSenderForTrack(MediaStreamTrackInterface* track); 328 FindSenderForTrack(MediaStreamTrackInterface* track);
330 std::vector<rtc::scoped_refptr<RtpReceiverInterface>>::iterator 329 std::vector<rtc::scoped_refptr<RtpReceiverInterface>>::iterator
331 FindReceiverForTrack(MediaStreamTrackInterface* track); 330 FindReceiverForTrack(MediaStreamTrackInterface* track);
332 331
333 TrackInfos* GetRemoteTracks(cricket::MediaType media_type); 332 TrackInfos* GetRemoteTracks(cricket::MediaType media_type);
334 TrackInfos* GetLocalTracks(cricket::MediaType media_type); 333 TrackInfos* GetLocalTracks(cricket::MediaType media_type);
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
387 // because its destruction fires signals (such as VoiceChannelDestroyed) 386 // because its destruction fires signals (such as VoiceChannelDestroyed)
388 // which will trigger some final actions in PeerConnection... 387 // which will trigger some final actions in PeerConnection...
389 rtc::scoped_ptr<WebRtcSession> session_; 388 rtc::scoped_ptr<WebRtcSession> session_;
390 // ... But stats_ depends on session_ so it should be destroyed even earlier. 389 // ... But stats_ depends on session_ so it should be destroyed even earlier.
391 rtc::scoped_ptr<StatsCollector> stats_; 390 rtc::scoped_ptr<StatsCollector> stats_;
392 }; 391 };
393 392
394 } // namespace webrtc 393 } // namespace webrtc
395 394
396 #endif // TALK_APP_WEBRTC_PEERCONNECTION_H_ 395 #endif // TALK_APP_WEBRTC_PEERCONNECTION_H_
OLDNEW
« no previous file with comments | « talk/app/webrtc/objctests/RTCPeerConnectionSyncObserver.m ('k') | talk/app/webrtc/peerconnection.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698