Enable peerconnection to use the NSS RNG.

content/renderer/media/media_stream_dependency_factory.cc

 // 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.
 
 #include "content/renderer/media/media_stream_dependency_factory.h"
 
 #include <vector>
 
 #include "base/synchronization/waitable_event.h"
 #include "base/utf_string_conversions.h"
 #include "content/renderer/media/media_stream_source_extra_data.h"
 #include "content/renderer/media/peer_connection_handler_jsep.h"
 #include "content/renderer/media/rtc_peer_connection_handler.h"
 #include "content/renderer/media/rtc_video_capturer.h"
 #include "content/renderer/media/video_capture_impl_manager.h"
 #include "content/renderer/media/webrtc_audio_device_impl.h"
 #include "content/renderer/media/webrtc_uma_histograms.h"
 #include "content/renderer/p2p/ipc_network_manager.h"
 #include "content/renderer/p2p/ipc_socket_factory.h"
 #include "content/renderer/p2p/port_allocator.h"
 #include "jingle/glue/thread_wrapper.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebFrame.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebMediaStreamComponent.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebMediaStreamDescriptor.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebMediaStreamSource.h"
 
+#if defined(USE_NSS)

[wtc, 2012/10/11 18:22:16]

The two ifdefs you added should be either

  #if !defined(USE_OPENSSL)

or

  #if defined(USE_NSS) || defined(OS_MACOSX) || defined(OS_WIN)

The former is more concise.

USE_NSS is only relevent to Linux (including Chrome OS)
and Android. This is why defined(OS_MACOSX) and defined(OS_WIN)
are necessary if you check defined(USE_NSS). I understand
this is complicated. Sorry about that.

[ekr, 2012/10/11 18:29:49]

I wonder if what you want is SSL_USE_OPENSSL and SSL_USE_NSS (in libjingle's
sslconfig.h). We could then arrange for these to be set properly by the platform

[Ronghua Wu (Left Chromium), 2012/10/11 21:13:53]

Done.

+#include "net/socket/nss_ssl_util.h"
+#endif
+
 class P2PPortAllocatorFactory : public webrtc::PortAllocatorFactoryInterface {
  public:
   P2PPortAllocatorFactory(
       content::P2PSocketDispatcher* socket_dispatcher,
       talk_base::NetworkManager* network_manager,
       talk_base::PacketSocketFactory* socket_factory,
       WebKit::WebFrame* web_frame)
       : socket_dispatcher_(socket_dispatcher),
         network_manager_(network_manager),
         socket_factory_(socket_factory),
@@ skipping 308 matching lines @@
         base::Unretained(this),
         &event));
     event.Wait();
   }
 
   if (!socket_factory_.get()) {
     socket_factory_.reset(
         new content::IpcPacketSocketFactory(p2p_socket_dispatcher_));
   }
 
+#if defined(USE_NSS)
+  // Init NSS, which will be needed by PeerConnection.
+  net::EnsureNSSSSLInit();
+#endif
+
   if (!CreatePeerConnectionFactory()) {
     LOG(ERROR) << "Could not create PeerConnection factory";
     return false;
   }
   return true;
 }
 
 void MediaStreamDependencyFactory::CleanupPeerConnectionFactory() {
   pc_factory_ = NULL;
   if (network_manager_) {
     // The network manager needs to free its resources on the thread they were
     // created, which is the worked thread.
     if (chrome_worker_thread_.IsRunning()) {
       chrome_worker_thread_.message_loop()->PostTask(FROM_HERE, base::Bind(
           &MediaStreamDependencyFactory::DeleteIpcNetworkManager,
           base::Unretained(this)));
       // Stopping the thread will wait until all tasks have been
       // processed before returning. We wait for the above task to finish before
       // letting the the function continue to avoid any potential race issues.
       chrome_worker_thread_.Stop();
     } else {
       NOTREACHED() << "Worker thread not running.";
     }
   }
 }