Do FilteringNetworkManager::CheckPermission from new Initialize method.

content/renderer/p2p/port_allocator.h

Index: content/renderer/p2p/port_allocator.h
diff --git a/content/renderer/p2p/port_allocator.h b/content/renderer/p2p/port_allocator.h
index 71a1f6111922ee93e3a3e785e8506b5f7dac4b70..f686b39643e4ded4c49106a74d0095caae073c31 100644
--- a/content/renderer/p2p/port_allocator.h
+++ b/content/renderer/p2p/port_allocator.h
@@ -45,6 +45,9 @@ class P2PPortAllocator : public cricket::BasicPortAllocator {
                    const GURL& origin);
   ~P2PPortAllocator() override;
 
+  // Will also initialize the network manager passed into the constructor.
+  void Initialize() override;

[Sergey Ulanov, 2016/07/07 18:09:08]

Who is supposed to call this method? I don't see it being called anywhere right
now.

[Taylor_Brandstetter, 2016/07/07 21:30:28]

It's called by PeerConnection:
https://cs.chromium.org/chromium/src/third_party/webrtc/api/peerconnection.cc...

This is so that PeerConnection can handle the thread-specific initialization,
meaning PeerConnectionDependencyFactory doesn't need to.

+
  private:
   std::unique_ptr<rtc::NetworkManager> network_manager_;
   scoped_refptr<P2PSocketDispatcher> socket_dispatcher_;