Pulling user experiment code from BrowserDistribution to a new class.

chrome/installer/util/user_experiment.h

+// Copyright 2013 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.
+//
+// This files declares a class that contains methods and data to conduct
+// for user expeirments.
+
+#ifndef CHROME_INSTALLER_UTIL_USER_EXPERIMENT_H_
+#define CHROME_INSTALLER_UTIL_USER_EXPERIMENT_H_
+
+#include "base/string16.h"
+#include "chrome/installer/util/browser_distribution.h"
+#include "chrome/installer/util/util_constants.h"
+
+class BrowserDistribution;
+
+namespace base {
+class FilePath;
+}
+
+namespace installer {
+
+class Product;
+
+namespace user_experiment {

[grt (UTC plus 2), 2013/03/05 14:33:22]

looking at how this is used, i'm now of the mind that these types and functions
should just be in the installer namespace. there's no risk of name collision as
far as i can tell, and seeing user_experiment doesn't really help the reader all
that much given that all functions have "Experiment" in their names. so, with
that, please nix the user_experiment namespace altogether unless someone
strongly objects.

[gab, 2013/03/05 14:58:00]

sgtm.

[huangs, 2013/03/05 16:48:34]

Done.  Also changing members of ToastUiFlags (just renamed from ToastUIflags) to
have prefix kToastUi*.

+
+// Flags to control what to show in the UserExperiment dialog.
+enum ToastUIflags {
+  kUninstall          = 1 << 0,  // Uninstall radio button.
+  kDontBugMeAsButton  = 1 << 1,  // This is a button, not a radio button.
+  kWhyLink            = 1 << 2,  // Has the 'why I am seeing this' link.
+  kMakeDefault        = 1 << 3,  // Has the 'make it default' checkbox.
+};
+
+// A struct for communicating what a UserExperiment contains. In these
+// experiments we show toasts to the user if they are inactive for a certain
+// amount of time.
+struct ExperimentDetails {
+  string16 prefix;      // The experiment code prefix for this experiment,
+                        // also known as the 'TV' part in 'TV80'.
+  int flavor;           // The flavor index for this experiment.
+  int heading;          // The heading resource ID to use for this experiment.
+  int flags;            // See ToastUIFlags above.
+  int control_group;    // Size of the control group (in percentages). Control
+                        // group is the group that qualifies for the
+                        // experiment but does not participate.
+};
+
+// Creates the experiment details for a given language-brand combo.
+// If |flavor| is -1, then a flavor will be selected at random. |experiment|
+// is the struct you want to write the experiment information to.
+// Returns false if no experiment details could be gathered.
+bool CreateExperimentDetails(int flavor,
+                             ExperimentDetails* experiment);
+
+// After an install or upgrade the user might qualify to participate in an
+// experiment. This function determines if the user qualifies and if so it
+// sets the wheels in motion or in simple cases does the experiment itself.
+void LaunchUserExperiment(const base::FilePath& setup_path,
+                          InstallStatus status,
+                          const Product& product,
+                          bool system_level);
+
+// The user has qualified for the inactive user toast experiment and this
+// function just performs it.
+void InactiveUserToastExperiment(BrowserDistribution* dist,
+                                 int flavor,
+                                 const string16& experiment_group,
+                                 const Product& installation,
+                                 const base::FilePath& application_path);
+
+}  // namespace user_experiment
+
+}  // namespace installer
+
+#endif  // CHROME_INSTALLER_UTIL_USER_EXPERIMENT_H_