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

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

Issue 1647483004: 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 308 matching lines...) Expand 10 before | Expand all | Expand 10 after
319 319
320 // Notifications from WebRtcSession relating to BaseChannels. 320 // Notifications from WebRtcSession relating to BaseChannels.
321 void OnVoiceChannelDestroyed(); 321 void OnVoiceChannelDestroyed();
322 void OnVideoChannelDestroyed(); 322 void OnVideoChannelDestroyed();
323 void OnDataChannelCreated(); 323 void OnDataChannelCreated();
324 void OnDataChannelDestroyed(); 324 void OnDataChannelDestroyed();
325 // Called when the cricket::DataChannel receives a message indicating that a 325 // Called when the cricket::DataChannel receives a message indicating that a
326 // webrtc::DataChannel should be opened. 326 // webrtc::DataChannel should be opened.
327 void OnDataChannelOpenMessage(const std::string& label, 327 void OnDataChannelOpenMessage(const std::string& label,
328 const InternalDataChannelInit& config); 328 const InternalDataChannelInit& config);
329 void OnFirstMediaPacketReceived();
330 329
331 RtpSenderInterface* FindSenderById(const std::string& id); 330 RtpSenderInterface* FindSenderById(const std::string& id);
332 331
333 std::vector<rtc::scoped_refptr<RtpSenderInterface>>::iterator 332 std::vector<rtc::scoped_refptr<RtpSenderInterface>>::iterator
334 FindSenderForTrack(MediaStreamTrackInterface* track); 333 FindSenderForTrack(MediaStreamTrackInterface* track);
335 std::vector<rtc::scoped_refptr<RtpReceiverInterface>>::iterator 334 std::vector<rtc::scoped_refptr<RtpReceiverInterface>>::iterator
336 FindReceiverForTrack(MediaStreamTrackInterface* track); 335 FindReceiverForTrack(MediaStreamTrackInterface* track);
337 336
338 TrackInfos* GetRemoteTracks(cricket::MediaType media_type); 337 TrackInfos* GetRemoteTracks(cricket::MediaType media_type);
339 TrackInfos* GetLocalTracks(cricket::MediaType media_type); 338 TrackInfos* GetLocalTracks(cricket::MediaType media_type);
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
392 // because its destruction fires signals (such as VoiceChannelDestroyed) 391 // because its destruction fires signals (such as VoiceChannelDestroyed)
393 // which will trigger some final actions in PeerConnection... 392 // which will trigger some final actions in PeerConnection...
394 rtc::scoped_ptr<WebRtcSession> session_; 393 rtc::scoped_ptr<WebRtcSession> session_;
395 // ... But stats_ depends on session_ so it should be destroyed even earlier. 394 // ... But stats_ depends on session_ so it should be destroyed even earlier.
396 rtc::scoped_ptr<StatsCollector> stats_; 395 rtc::scoped_ptr<StatsCollector> stats_;
397 }; 396 };
398 397
399 } // namespace webrtc 398 } // namespace webrtc
400 399
401 #endif // TALK_APP_WEBRTC_PEERCONNECTION_H_ 400 #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