Check media permissions (mic/camera) before exposing local addresses to WebRTC.

content/renderer/p2p/port_allocator.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_RENDERER_P2P_PORT_ALLOCATOR_H_
 #define CONTENT_RENDERER_P2P_PORT_ALLOCATOR_H_
 
+#include "base/memory/ref_counted.h"
 #include "third_party/webrtc/p2p/client/basicportallocator.h"
 #include "url/gurl.h"
 
 namespace content {
 
 class P2PPortAllocatorSession;
 class P2PSocketDispatcher;
 
 class P2PPortAllocator : public cricket::BasicPortAllocator {
  public:
@@ skipping 19 matching lines @@
 
     // Enable non-proxied UDP-based transport when set to true. When set to
     // false, it effectively disables all UDP traffic until UDP-supporting proxy
     // RETURN is available in the future.
     bool enable_nonproxied_udp = true;
 
     // Disable binding to individual NICs when set to false.
     bool enable_multiple_routes = true;
   };
 
-  P2PPortAllocator(P2PSocketDispatcher* socket_dispatcher,
-                   rtc::NetworkManager* network_manager,
-                   rtc::PacketSocketFactory* socket_factory,
-                   const Config& config,
-                   const GURL& origin);
+  struct Params {

[Sergey Ulanov, 2015/09/21 22:12:59]

Do you really need to add this? Does it need to be in this CL?

[guoweis_left_chromium, 2015/09/22 17:56:56]

Done.

+    Params(const scoped_refptr<P2PSocketDispatcher>& socket_dispatcher,
+           rtc::PacketSocketFactory* socket_factory,
+           const GURL& origin,
+           const Config& config);
+    ~Params();
+    const scoped_refptr<P2PSocketDispatcher> socket_dispatcher;
+    rtc::PacketSocketFactory* socket_factory;
+    GURL requesting_origin;
+    Config config;
+  };
+
+  P2PPortAllocator(const Params& params,
+                   scoped_ptr<rtc::NetworkManager> network_manager);
   ~P2PPortAllocator() override;
 
   cricket::PortAllocatorSession* CreateSessionInternal(
       const std::string& content_name,
       int component,
       const std::string& ice_username_fragment,
       const std::string& ice_password) override;
 
  private:
   friend class P2PPortAllocatorSession;
-
-  P2PSocketDispatcher* socket_dispatcher_;
+  scoped_ptr<rtc::NetworkManager> network_manager_;
+  scoped_refptr<P2PSocketDispatcher> socket_dispatcher_;
   Config config_;
   GURL origin_;
 
   DISALLOW_COPY_AND_ASSIGN(P2PPortAllocator);
 };
 
 class P2PPortAllocatorSession : public cricket::BasicPortAllocatorSession {
  public:
   P2PPortAllocatorSession(
       P2PPortAllocator* allocator,
       const std::string& content_name,
       int component,
       const std::string& ice_username_fragment,
       const std::string& ice_password);
   ~P2PPortAllocatorSession() override;
 
  protected:
   // Overrides for cricket::BasicPortAllocatorSession.
   void GetPortConfigurations() override;
 
  private:
   P2PPortAllocator* allocator_;
 
   DISALLOW_COPY_AND_ASSIGN(P2PPortAllocatorSession);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_P2P_PORT_ALLOCATOR_H_