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

Side by Side Diff: webrtc/api/webrtcsession.h

Issue 1999853002: Forward the SignalFirstPacketReceived to RtpReceiver. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Add media type to BaseChannel. Created 4 years, 7 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 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright 2012 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 293 matching lines...) Expand 10 before | Expand all | Expand 10 after
304 304
305 cricket::DataChannelType data_channel_type() const; 305 cricket::DataChannelType data_channel_type() const;
306 306
307 bool IceRestartPending(const std::string& content_name) const; 307 bool IceRestartPending(const std::string& content_name) const;
308 308
309 // Called when an RTCCertificate is generated or retrieved by 309 // Called when an RTCCertificate is generated or retrieved by
310 // WebRTCSessionDescriptionFactory. Should happen before setLocalDescription. 310 // WebRTCSessionDescriptionFactory. Should happen before setLocalDescription.
311 void OnCertificateReady( 311 void OnCertificateReady(
312 const rtc::scoped_refptr<rtc::RTCCertificate>& certificate); 312 const rtc::scoped_refptr<rtc::RTCCertificate>& certificate);
313 void OnDtlsSetupFailure(cricket::BaseChannel*, bool rtcp); 313 void OnDtlsSetupFailure(cricket::BaseChannel*, bool rtcp);
314 // Called when the channel received the first packet.
315 void OnChannelFirstPacketReceived(cricket::BaseChannel*);
314 316
315 // For unit test. 317 // For unit test.
316 bool waiting_for_certificate_for_testing() const; 318 bool waiting_for_certificate_for_testing() const;
317 const rtc::scoped_refptr<rtc::RTCCertificate>& certificate_for_testing(); 319 const rtc::scoped_refptr<rtc::RTCCertificate>& certificate_for_testing();
318 320
319 void set_metrics_observer( 321 void set_metrics_observer(
320 webrtc::MetricsObserverInterface* metrics_observer) { 322 webrtc::MetricsObserverInterface* metrics_observer) {
321 metrics_observer_ = metrics_observer; 323 metrics_observer_ = metrics_observer;
322 } 324 }
323 325
(...skipping 197 matching lines...) Expand 10 before | Expand all | Expand 10 after
521 cricket::AudioOptions audio_options_; 523 cricket::AudioOptions audio_options_;
522 cricket::VideoOptions video_options_; 524 cricket::VideoOptions video_options_;
523 MetricsObserverInterface* metrics_observer_; 525 MetricsObserverInterface* metrics_observer_;
524 526
525 // Declares the bundle policy for the WebRTCSession. 527 // Declares the bundle policy for the WebRTCSession.
526 PeerConnectionInterface::BundlePolicy bundle_policy_; 528 PeerConnectionInterface::BundlePolicy bundle_policy_;
527 529
528 // Declares the RTCP mux policy for the WebRTCSession. 530 // Declares the RTCP mux policy for the WebRTCSession.
529 PeerConnectionInterface::RtcpMuxPolicy rtcp_mux_policy_; 531 PeerConnectionInterface::RtcpMuxPolicy rtcp_mux_policy_;
530 532
533 bool has_received_video_packet_ = false;
534 bool has_received_audio_packet_ = false;
pthatcher1 2016/06/08 17:35:56 You don't need the "has_" prefix.
535
531 RTC_DISALLOW_COPY_AND_ASSIGN(WebRtcSession); 536 RTC_DISALLOW_COPY_AND_ASSIGN(WebRtcSession);
532 }; 537 };
533 } // namespace webrtc 538 } // namespace webrtc
534 539
535 #endif // WEBRTC_API_WEBRTCSESSION_H_ 540 #endif // WEBRTC_API_WEBRTCSESSION_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698