Pulling user experiment code from BrowserDistribution to a new class.

chrome/installer/util/chrome_browser_operations.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 "chrome/installer/util/chrome_browser_operations.h"
 
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "base/string_util.h"
 #include "chrome/installer/util/browser_distribution.h"
 #include "chrome/installer/util/channel_info.h"
 #include "chrome/installer/util/helper.h"
 #include "chrome/installer/util/install_util.h"
 #include "chrome/installer/util/master_preferences.h"
 #include "chrome/installer/util/master_preferences_constants.h"
 #include "chrome/installer/util/shell_util.h"
+#include "chrome/installer/util/user_experiment.h"
 #include "chrome/installer/util/util_constants.h"
 
 namespace installer {
 
 void ChromeBrowserOperations::ReadOptions(
     const MasterPreferences& prefs,
     std::set<std::wstring>* options) const {
   DCHECK(options);
 
   bool pref_value;
@@ skipping 100 matching lines @@
     bool is_per_user_install =
         InstallUtil::IsPerUserInstall(target_exe.value().c_str());
     properties->set_app_id(
         ShellUtil::GetBrowserModelId(dist, is_per_user_install));
   }
 
   if (!properties->has_description())
     properties->set_description(dist->GetAppDescription());
 }
 
+void ChromeBrowserOperations::LaunchUserExperiment(
+    const base::FilePath& setup_path,
+    const std::set<std::wstring>& options,
+    InstallStatus status,
+    bool system_level) const {
+  BrowserDistribution* dist = BrowserDistribution::GetSpecificDistribution(

[grt (UTC plus 2), 2013/03/12 19:36:18]

please move into the if(){} block in product.cc as per my original request.

[huangs, 2013/03/14 17:20:45]

Done.  Sorry, misread!

+      BrowserDistribution::CHROME_BROWSER);
+  VLOG(1) << "LaunchUserExperiment status: " << status << " product: "
+          << dist->GetAppShortCutName()
+          << " system_level: " << system_level;
+
+  CommandLine base_command(setup_path);
+  AppendProductFlags(options, &base_command);
+  installer::LaunchBrowserUserExperiment(base_command, status, system_level);
+}
+
 }  // namespace installer