Use a different algorithm with the low entropy source for field trials.

chrome_frame/test/run_all_unittests.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 <atlbase.h>
 
 #include "base/command_line.h"
 #include "base/metrics/field_trial.h"
 #include "base/process_util.h"
 #include "base/test/test_suite.h"
@@ skipping 36 matching lines @@
 int main(int argc, char **argv) {
   base::win::ScopedCOMInitializer com_initializer;
   ScopedChromeFrameRegistrar::RegisterAndExitProcessIfDirected();
   base::EnableTerminationOnHeapCorruption();
   base::PlatformThread::SetName("ChromeFrame tests");
 
   _set_purecall_handler(PureCall);
 
   // Set up a FieldTrialList to keep any field trials we have going in
   // Chrome Frame happy.
-  base::FieldTrialList field_trial_list("42");
+  base::FieldTrialList field_trial_list(NULL);

[robertshield, 2012/08/16 21:18:01]

I disapprove of this change aesthetically, though pragmatically it looks right.

[Alexei Svitkine (slow), 2012/08/17 16:42:16]

In fact, I had to change this back to not pass NULL since it caused some
chrome_frame tests failures (I guess some chrome frame field trials require one
time randomization).

 
   base::TestSuite test_suite(argc, argv);
 
   SetConfigBool(kChromeFrameHeadlessMode, true);
   SetConfigBool(kChromeFrameAccessibleMode, true);
 
   base::ProcessHandle crash_service = NULL;
   google_breakpad::scoped_ptr<google_breakpad::ExceptionHandler> breakpad;
 
   if (!CommandLine::ForCurrentProcess()->HasSwitch(kNoCrashService)) {
@@ skipping 34 matching lines @@
     ret = test_suite.Run();
   }
 
   DeleteConfigValue(kChromeFrameHeadlessMode);
   DeleteConfigValue(kChromeFrameAccessibleMode);
 
   if (crash_service)
     base::KillProcess(crash_service, 0, false);
   return ret;
 }