NSS: don't set environment variables when multi-threaded.

chrome/app/chrome_main.cc

Index: chrome/app/chrome_main.cc
diff --git a/chrome/app/chrome_main.cc b/chrome/app/chrome_main.cc
index b13944cf0bb8ed9c518a55e64b35e46ae971c94e..64530e6eb8862911b9e3145ff7d856f5c167e651 100644
--- a/chrome/app/chrome_main.cc
+++ b/chrome/app/chrome_main.cc
@@ -13,6 +13,7 @@
 #include "base/message_loop.h"
 #include "base/metrics/stats_counters.h"
 #include "base/metrics/stats_table.h"
+#include "base/nss_util.h"
 #include "base/path_service.h"
 #include "base/process_util.h"
 #include "base/string_number_conversions.h"
@@ -714,6 +715,8 @@ int ChromeMain(int argc, char** argv) {
   app::RegisterPathProvider();
   chrome::RegisterPathProvider();
 
+  base::EarlyNSSInit();

[wtc, 2011/01/13 00:07:02]

Nit: since EarlyNSSInit doesn't use the Chrome path provider, it seems
that we can call EarlyNSSInit earlier, immediately after the SecupCRT call
at line 712.

At least, we should not call EarlyNSSInit in the middle of the code block
that sets up the Chrome path provider (lines 714-724).

+
   // Notice a user data directory override if any
   const FilePath user_data_dir =
       command_line.GetSwitchValuePath(switches::kUserDataDir);