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

Unified Diff: webrtc/p2p/base/portallocator.h

Issue 2124283003: Fixing inconsistency with behavior of `ClearGettingPorts`. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: webrtc/p2p/base/portallocator.h
diff --git a/webrtc/p2p/base/portallocator.h b/webrtc/p2p/base/portallocator.h
index 152474d4f4476b5fc3060bb092e9f83cebd9aac3..f11ed085ee398ba22bac9a0d084cf6fc387c4990 100644
--- a/webrtc/p2p/base/portallocator.h
+++ b/webrtc/p2p/base/portallocator.h
@@ -158,14 +158,22 @@ class PortAllocatorSession : public sigslot::has_slots<> {
// Default filter should be CF_ALL.
virtual void SetCandidateFilter(uint32_t filter) = 0;
- // Starts gathering STUN and Relay configurations.
+ // Starts gathering ports and ICE candidates.
virtual void StartGettingPorts() = 0;
- // Completely stops the gathering process and will not start new ones.
+ // Completely stops gathering. Will not gather again unless StartGettingPorts
+ // is called again.
virtual void StopGettingPorts() = 0;
// Whether the session is actively getting ports.
virtual bool IsGettingPorts() = 0;
- // ClearGettingPorts and IsCleared are used by continual gathering.
- // Only stops the existing gathering process but may start new ones if needed.
+
+ //
+ // NOTE: The group of methods below is only used for continual gathering.
+ //
+
+ // ClearGettingPorts should have the same immediate effect as
+ // StopGettingPorts, but if the implementation supports continual gathering,
+ // ClearGettingPorts allows additional ports/candidates to be gathered if the
+ // network conditions change.
virtual void ClearGettingPorts() = 0;
// Whether it is in the state where the existing gathering process is stopped,
// but new ones may be started (basically after calling ClearGettingPorts).

Powered by Google App Engine
This is Rietveld 408576698