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

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

Issue 1453813005: Fixing some issues with ICE restart signaling. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 5 years, 1 month 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 276 matching lines...) Expand 10 before | Expand all | Expand 10 after
287 virtual bool GetLocalCertificate( 287 virtual bool GetLocalCertificate(
288 const std::string& transport_name, 288 const std::string& transport_name,
289 rtc::scoped_refptr<rtc::RTCCertificate>* certificate); 289 rtc::scoped_refptr<rtc::RTCCertificate>* certificate);
290 290
291 // Caller owns returned certificate 291 // Caller owns returned certificate
292 virtual bool GetRemoteSSLCertificate(const std::string& transport_name, 292 virtual bool GetRemoteSSLCertificate(const std::string& transport_name,
293 rtc::SSLCertificate** cert); 293 rtc::SSLCertificate** cert);
294 294
295 cricket::DataChannelType data_channel_type() const; 295 cricket::DataChannelType data_channel_type() const;
296 296
297 bool IceRestartPending() const; 297 bool AudioIceRestartPending() const;
298 298 bool VideoIceRestartPending() const;
299 void ResetIceRestartLatch(); 299 bool DataIceRestartPending() const;
300 300
301 // Called when an RTCCertificate is generated or retrieved by 301 // Called when an RTCCertificate is generated or retrieved by
302 // WebRTCSessionDescriptionFactory. Should happen before setLocalDescription. 302 // WebRTCSessionDescriptionFactory. Should happen before setLocalDescription.
303 void OnCertificateReady( 303 void OnCertificateReady(
304 const rtc::scoped_refptr<rtc::RTCCertificate>& certificate); 304 const rtc::scoped_refptr<rtc::RTCCertificate>& certificate);
305 void OnDtlsSetupFailure(cricket::BaseChannel*, bool rtcp); 305 void OnDtlsSetupFailure(cricket::BaseChannel*, bool rtcp);
306 306
307 // For unit test. 307 // For unit test.
308 bool waiting_for_certificate_for_testing() const; 308 bool waiting_for_certificate_for_testing() const;
309 const rtc::scoped_refptr<rtc::RTCCertificate>& certificate_for_testing(); 309 const rtc::scoped_refptr<rtc::RTCCertificate>& certificate_for_testing();
(...skipping 177 matching lines...) Expand 10 before | Expand all | Expand 10 after
487 bool older_version_remote_peer_; 487 bool older_version_remote_peer_;
488 bool dtls_enabled_; 488 bool dtls_enabled_;
489 // Specifies which kind of data channel is allowed. This is controlled 489 // Specifies which kind of data channel is allowed. This is controlled
490 // by the chrome command-line flag and constraints: 490 // by the chrome command-line flag and constraints:
491 // 1. If chrome command-line switch 'enable-sctp-data-channels' is enabled, 491 // 1. If chrome command-line switch 'enable-sctp-data-channels' is enabled,
492 // constraint kEnableDtlsSrtp is true, and constaint kEnableRtpDataChannels is 492 // constraint kEnableDtlsSrtp is true, and constaint kEnableRtpDataChannels is
493 // not set or false, SCTP is allowed (DCT_SCTP); 493 // not set or false, SCTP is allowed (DCT_SCTP);
494 // 2. If constraint kEnableRtpDataChannels is true, RTP is allowed (DCT_RTP); 494 // 2. If constraint kEnableRtpDataChannels is true, RTP is allowed (DCT_RTP);
495 // 3. If both 1&2 are false, data channel is not allowed (DCT_NONE). 495 // 3. If both 1&2 are false, data channel is not allowed (DCT_NONE).
496 cricket::DataChannelType data_channel_type_; 496 cricket::DataChannelType data_channel_type_;
497 rtc::scoped_ptr<IceRestartAnswerLatch> ice_restart_latch_; 497 rtc::scoped_ptr<IceRestartAnswerLatch> audio_ice_restart_latch_;
498 rtc::scoped_ptr<IceRestartAnswerLatch> video_ice_restart_latch_;
499 rtc::scoped_ptr<IceRestartAnswerLatch> data_ice_restart_latch_;
498 500
499 rtc::scoped_ptr<WebRtcSessionDescriptionFactory> 501 rtc::scoped_ptr<WebRtcSessionDescriptionFactory>
500 webrtc_session_desc_factory_; 502 webrtc_session_desc_factory_;
501 503
502 // Member variables for caching global options. 504 // Member variables for caching global options.
503 cricket::AudioOptions audio_options_; 505 cricket::AudioOptions audio_options_;
504 cricket::VideoOptions video_options_; 506 cricket::VideoOptions video_options_;
505 MetricsObserverInterface* metrics_observer_; 507 MetricsObserverInterface* metrics_observer_;
506 508
507 // Declares the bundle policy for the WebRTCSession. 509 // Declares the bundle policy for the WebRTCSession.
508 PeerConnectionInterface::BundlePolicy bundle_policy_; 510 PeerConnectionInterface::BundlePolicy bundle_policy_;
509 511
510 // Declares the RTCP mux policy for the WebRTCSession. 512 // Declares the RTCP mux policy for the WebRTCSession.
511 PeerConnectionInterface::RtcpMuxPolicy rtcp_mux_policy_; 513 PeerConnectionInterface::RtcpMuxPolicy rtcp_mux_policy_;
512 514
513 RTC_DISALLOW_COPY_AND_ASSIGN(WebRtcSession); 515 RTC_DISALLOW_COPY_AND_ASSIGN(WebRtcSession);
514 }; 516 };
515 } // namespace webrtc 517 } // namespace webrtc
516 518
517 #endif // TALK_APP_WEBRTC_WEBRTCSESSION_H_ 519 #endif // TALK_APP_WEBRTC_WEBRTCSESSION_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698