Add UMA logging of search engine selection position when the search engine lo...

chrome/browser/browser_main.cc

 // Copyright (c) 2010 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/browser/browser_main.h"
 
 #include <algorithm>
 #include <string>
 #include <vector>
 
@@ skipping 1388 matching lines @@
 #endif
 
 #if defined(OS_LINUX) && !defined(OS_CHROMEOS)
       // On Linux, the running exe will be updated if an upgrade becomes
       // available while the browser is running.  We need to save the last
       // modified time of the exe, so we can compare to determine if there is
       // an upgrade while the browser is kept alive by a persistent extension.
       Upgrade::SaveLastModifiedTimeOfExe();
 #endif
 
-      // Record now as the last succesful chrome start.
+      // Record now as the last successful chrome start.
       GoogleUpdateSettings::SetLastRunTime();
       // Call Recycle() here as late as possible, before going into the loop
       // because Start() will add things to it while creating the main window.
       if (pool)
         pool->Recycle();
       RunUIMessageLoop(browser_process.get());
     }
   }
 
 #if defined(OS_WIN)
   // If it's the first run, log the search engine chosen.  We wait until
   // shutdown because otherwise we can't be sure the user has finished
   // selecting a search engine through the dialog reached from the first run
   // bubble link.
   if (FirstRun::InSearchExperimentLocale() && record_search_engine) {
     const TemplateURL* default_search_engine =
         profile->GetTemplateURLModel()->GetDefaultSearchProvider();
+    // Record the search engine chosen.
     if (master_prefs.run_search_engine_experiment) {
       UMA_HISTOGRAM_ENUMERATION(
           "Chrome.SearchSelectExperiment",
           TemplateURLPrepopulateData::GetSearchEngineType(
+          default_search_engine),
+          TemplateURLPrepopulateData::SEARCH_ENGINE_MAX);
+      // If the selection has been randomized, also record the winner by slot.
+      if (master_prefs.randomize_search_engine_experiment) {
+        size_t engine_pos = profile->GetTemplateURLModel()->
+            GetSearchEngineDialogSlot();
+        if (engine_pos < 4) {
+          std::string experiment_type = "Chrome.SearchSelectExperimentSlot";
+          // Nicer in UMA if slots are 1-based.
+          experiment_type.push_back('1' + engine_pos);
+          UMA_HISTOGRAM_ENUMERATION(
+              experiment_type,
+              TemplateURLPrepopulateData::GetSearchEngineType(
               default_search_engine),
-          TemplateURLPrepopulateData::SEARCH_ENGINE_MAX);
+              TemplateURLPrepopulateData::SEARCH_ENGINE_MAX);
+        } else {
+          NOTREACHED() << "Invalid search engine selection slot.";
+        }
+      }
     } else {
       UMA_HISTOGRAM_ENUMERATION(
           "Chrome.SearchSelectExempt",
           TemplateURLPrepopulateData::GetSearchEngineType(
               default_search_engine),
           TemplateURLPrepopulateData::SEARCH_ENGINE_MAX);
     }
   }
 #endif
 
   chrome_browser_net_websocket_experiment::WebSocketExperimentRunner::Stop();
 
   process_singleton.Cleanup();
 
   metrics->Stop();
 
   // browser_shutdown takes care of deleting browser_process, so we need to
   // release it.
   ignore_result(browser_process.release());
   browser_shutdown::Shutdown();
 
   return result_code;
 }