Add threaded compositing mode to field trial

chrome/browser/gpu_util.cc

Index: chrome/browser/gpu_util.cc
diff --git a/chrome/browser/gpu_util.cc b/chrome/browser/gpu_util.cc
index 10293ad9271f4987b4801617c2a774f415cdcd43..d586c5ee192515417c3948c34439fa2788d910b2 100644
--- a/chrome/browser/gpu_util.cc
+++ b/chrome/browser/gpu_util.cc
@@ -18,6 +18,7 @@
 #include "chrome/browser/gpu_blacklist.h"
 #include "chrome/common/chrome_version_info.h"
 #include "content/public/browser/gpu_data_manager.h"
+#include "content/public/common/content_constants.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/common/gpu_info.h"
 
@@ -166,13 +167,17 @@ int GetGpuBlacklistHistogramValueWin(GpuFeatureStatus status) {
 }
 #endif  // OS_WIN
 
+bool InForceThreadedCompositingModeTrial() {
+  base::FieldTrial* trial =
+      base::FieldTrialList::Find(content::kGpuCompositingFieldTrialName);
+  return trial && trial->group_name() ==
+      content::kGpuCompositingFieldTrialThreadEnabledName;
+}
+
 }  // namespace
 
 namespace gpu_util {
 
-const char kForceCompositingModeFieldTrialName[] = "ForceCompositingMode";
-const char kFieldTrialEnabledName[] = "enabled";
-
 void InitializeForceCompositingModeFieldTrial() {
 // Enable the field trial only on desktop OS's.
 #if !(defined(OS_WIN) || defined(OS_MACOSX) || defined(OS_LINUX))
@@ -196,32 +201,40 @@ void InitializeForceCompositingModeFieldTrial() {
   // Don't activate the field trial if force-compositing-mode has been
   // explicitly disabled from the command line.
   if (CommandLine::ForCurrentProcess()->HasSwitch(
-          switches::kDisableForceCompositingMode))
+          switches::kDisableForceCompositingMode) ||
+      CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kDisableThreadedCompositing))
     return;
 
-  const base::FieldTrial::Probability kDivisor = 100;
+  const base::FieldTrial::Probability kDivisor = 99;

[Ilya Sherman, 2012/07/20 05:36:05]

Optional nit: Perhaps set this to 3 rather than to 99?  I think 1/3 is clearer
than 33/99, since it's not a percentage either way.

[jbates, 2012/07/23 17:47:00]

Done.

   scoped_refptr<base::FieldTrial> trial(
     base::FieldTrialList::FactoryGetFieldTrial(
-        kForceCompositingModeFieldTrialName, kDivisor,
+        content::kGpuCompositingFieldTrialName, kDivisor,
         "disable", 2012, 12, 31, NULL));
 
   // Produce the same result on every run of this client.
   trial->UseOneTimeRandomization();
-  // 50% probability of being in the enabled group.
-  const base::FieldTrial::Probability kEnableProbability = 50;
+  // (33/99)% probability of being in the enabled or thread group.
+  const base::FieldTrial::Probability kEnableProbability = 33;
   int enable_group = trial->AppendGroup(
-      kFieldTrialEnabledName, kEnableProbability);
+      content::kGpuCompositingFieldTrialEnabledName, kEnableProbability);
+  int thread_group = trial->AppendGroup(
+      content::kGpuCompositingFieldTrialThreadEnabledName, kEnableProbability);
 
   bool enabled = (trial->group() == enable_group);
+  bool thread = (trial->group() == thread_group);
   UMA_HISTOGRAM_BOOLEAN("GPU.InForceCompositingModeFieldTrial", enabled);
+  UMA_HISTOGRAM_BOOLEAN("GPU.InCompositorThreadFieldTrial", thread);
 }
 
-bool InForceCompositingModeTrial() {
+bool InForceCompositingModeOrThreadTrial() {
   base::FieldTrial* trial =
-      base::FieldTrialList::Find(kForceCompositingModeFieldTrialName);
+      base::FieldTrialList::Find(content::kGpuCompositingFieldTrialName);
   if (!trial)
     return false;
-  return trial->group_name() == kFieldTrialEnabledName;
+  return trial->group_name() == content::kGpuCompositingFieldTrialEnabledName ||
+         trial->group_name() ==
+             content::kGpuCompositingFieldTrialThreadEnabledName;
 }
 
 GpuFeatureType StringToGpuFeatureType(const std::string& feature_string) {
@@ -372,14 +385,15 @@ Value* GetFeatureStatus() {
              (flags & content::GPU_FEATURE_TYPE_ACCELERATED_COMPOSITING)))
           status += "_readback";
         bool has_thread =
-            command_line.HasSwitch(switches::kEnableThreadedCompositing) &&
-            !command_line.HasSwitch(switches::kDisableThreadedCompositing);
+            (command_line.HasSwitch(switches::kEnableThreadedCompositing) &&
+             !command_line.HasSwitch(switches::kDisableThreadedCompositing)) ||
+            InForceThreadedCompositingModeTrial();
         if (kGpuFeatureInfo[i].name == "compositing") {
           bool force_compositing =
               (command_line.HasSwitch(switches::kForceCompositingMode) &&
                !command_line.HasSwitch(
                    switches::kDisableForceCompositingMode)) ||
-              InForceCompositingModeTrial();
+              InForceCompositingModeOrThreadTrial();
           if (force_compositing)
             status += "_force";
           if (has_thread)