Fixing the behavior of the candidate filter with pooled candidates.

webrtc/p2p/base/portallocator.h

Index: webrtc/p2p/base/portallocator.h
diff --git a/webrtc/p2p/base/portallocator.h b/webrtc/p2p/base/portallocator.h
index 879657081a5372f1248178964523c0d5ed2d1539..f05947d7c22f48bcb5c92cf6a7a7d6691390accc 100644
--- a/webrtc/p2p/base/portallocator.h
+++ b/webrtc/p2p/base/portallocator.h
@@ -142,6 +142,13 @@ class PortAllocatorSession : public sigslot::has_slots<> {
   const std::string& ice_pwd() const { return ice_pwd_; }
   bool pooled() const { return ice_ufrag_.empty(); }
 
+  // Setting this filter should affect not only candidates gathered in the
+  // future, but candidates returned by ReadyCandidates() and ports returned
+  // by ReadyPorts().
+  //
+  // Default filter should be CF_ALL.
+  virtual void SetCandidateFilter(uint32_t filter) = 0;
+
   // Starts gathering STUN and Relay configurations.
   virtual void StartGettingPorts() = 0;
   virtual void StopGettingPorts() = 0;
@@ -301,7 +308,6 @@ class PortAllocator : public sigslot::has_slots<> {
 
   uint32_t candidate_filter() { return candidate_filter_; }
   void set_candidate_filter(uint32_t filter) {
-    // TODO(mallinath) - Do transition check?

[Taylor Brandstetter, 2016/05/19 22:11:45]

Transition check now done (in BasicPortAllocatorSession)

     candidate_filter_ = filter;
   }