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

chrome/browser/metrics/metrics_service.h

Index: chrome/browser/metrics/metrics_service.h
===================================================================
--- chrome/browser/metrics/metrics_service.h	(revision 151948)
+++ chrome/browser/metrics/metrics_service.h	(working copy)
@@ -16,6 +16,7 @@
 #include "base/gtest_prod_util.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
+#include "base/metrics/field_trial.h"
 #include "base/process_util.h"
 #include "chrome/browser/metrics/metrics_log.h"
 #include "chrome/browser/metrics/tracking_synchronizer_observer.h"
@@ -38,6 +39,7 @@
 
 namespace base {
 class DictionaryValue;
+class EntropyProvider;

[Ilya Sherman, 2012/08/17 07:34:28]

Isn't this class defined as a subtype of the FieldTrial class?

[Alexei Svitkine (slow), 2012/08/17 14:08:59]

You're right. This got left over from a previous iteration where I had the
EntropyProvider as a standalone interface. Removed.

 class MessageLoopProxy;
 }
 
@@ -86,16 +88,21 @@
   // recording is not currently running.
   std::string GetClientId();
 
-  // Returns the preferred entropy source used to seed persistent activities
+  // Returns the preferred entropy provider used to seed persistent activities
   // based on whether or not metrics reporting will permitted on this client.
   // The caller must determine if metrics reporting will be enabled for this
-  // client and pass that state in as |reporting_will_be_enabled|. If
-  // |reporting_will_be_enabled| is true, this method returns the client ID
-  // concatenated with the low entropy source. Otherwise, this method just
-  // returns the low entropy source. Note that this reporting state can not be
-  // checked by reporting_active() because this method may need to be called
-  // before the MetricsService needs to be started.
-  std::string GetEntropySource(bool reporting_will_be_enabled);
+  // client and pass that state in as |reporting_will_be_enabled|.
+  //
+  // If |reporting_will_be_enabled| is true, this method returns an entropy
+  // provider that has a high source of entropy, partially based on the client
+  // ID. Otherwise, an entropy provider that is based on a low entropy source
+  // is returned.
+  //
+  // Note that this reporting state can not be checked by reporting_active()
+  // because this method may need to be called before the MetricsService needs
+  // to be started.
+  base::FieldTrial::EntropyProvider* GetEntropyProvider(
+      bool reporting_will_be_enabled);
 
   // Force the client ID to be generated. This is useful in case it's needed
   // before recording.