Supporting command line argument to force field trials

content/public/common/content_switches.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/public/common/content_switches.h"
 
 namespace switches {
 
 // By default, file:// URIs cannot read other file:// URIs. This is an
 // override for developers who need the old behavior for testing.
@@ skipping 351 matching lines @@
 // - Location platform support for Windows 7
 const char kExperimentalLocationFeatures[]  = "experimental-location-features";
 
 // Load NPAPI plugins from the specified directory.
 const char kExtraPluginDir[]                = "extra-plugin-dir";
 
 // If accelerated compositing is supported, always enter compositing mode for
 // the base layer even when compositing is not strictly required.
 const char kForceCompositingMode[]          = "force-compositing-mode";
 
-// Some field tests may rendomized in the browser, and the randomly selected
-// outcome needs to be propagated to the renderer.  For instance, this is used
+// Some field tests may be randomized in the browser, and the randomly selected

[jar (doing other things), 2012/04/06 01:41:29]

nit: "field tests" --> "field trials"

[MAD, 2012/04/11 02:38:08]

Done.

+// outcome needs to be propagated to the renderer. For instance, this is used
 // to modify histograms recorded in the renderer, or to get the renderer to
 // also set of its state (initialize, or not initialize components) to match the
-// experiment(s).
-// The argument is a string-ized list of experiment names, and the associated
-// value that was randomly selected.  In the recent implementetaion, the
-// persistent representation generated by field_trial.cc and later decoded, is a
-// list of name and value pairs, separated by slashes. See field trial.cc for
-// current details.
+// experiment(s). The option is also useful for forcing field trials when
+// testing changes locally. The argument is a list of name and value pairs,
+// separated by slashes. See FieldTrialList::CreateTrialsFromString() in
+// field_trial.h for details.
 const char kForceFieldTestNameAndValue[]    = "force-fieldtest";

[jar (doing other things), 2012/04/06 01:41:29]

Is there a reason why you didn't call this something more like:
force-fieldtrial

Perchance I am the one that wrote this code :-(.

In any case, it should probably be fieldtrial unless there is a compelling
reason not to be such.

[MAD, 2012/04/11 02:38:08]

Done.

 
 // Force renderer accessibility to be on instead of enabling it on demand when
 // a screen reader is detected. The disable-renderer-accessibility switch
 // overrides this if present.
 const char kForceRendererAccessibility[]    = "force-renderer-accessibility";
 
 // Extra command line options for launching the GPU process (normally used
 // for debugging). Use like renderer-cmd-prefix.
 const char kGpuLauncher[]                   = "gpu-launcher";
 
@@ skipping 268 matching lines @@
 extern const char kFlingTapSuppressMaxDown[] = "fling-tap-suppress-max-down";
 
 // Maximum time between mousedown and mouseup to be considered a tap.
 extern const char kFlingTapSuppressMaxGap[] = "fling-tap-suppress-max-gap";
 
 // Forces usage of the test compositor. Needed to run ui tests on bots.
 extern const char kTestCompositor[]         = "test-compositor";
 #endif
 
 }  // namespace switches