Rename some chrome.exe exports: SetCommandLine --> SetCommandLine2, SetExperimentList --> SetExperi…

chrome/app/breakpad_field_trial_win.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/app/breakpad_field_trial_win.h"
 
 #include "base/lazy_instance.h"
 #include "base/string_util.h"
 #include "base/string16.h"
 #include "base/stringprintf.h"
 #include "breakpad/src/client/windows/common/ipc_protocol.h"
 #include "chrome/app/breakpad_win.h"
 #include "chrome/common/child_process_logging.h"
 
-extern "C" void __declspec(dllexport) __cdecl SetExperimentList(
+extern "C" void __declspec(dllexport) __cdecl SetExperimentList2(

[Sigurður Ásgeirsson, 2012/05/23 21:30:31]

Please add a comment here explaining that it's critical to change the name if
the signature of this function changes.

[eroman, 2012/05/23 21:45:13]

Done.

       const wchar_t** experiment_strings, size_t experiment_strings_size) {
   // Make sure we were initialized before we start writing data
   if (breakpad_win::g_experiment_chunks_offset == 0)
     return;
 
   size_t num_chunks = 0;
   size_t current_experiment = 0;
   string16 current_chunk(google_breakpad::CustomInfoEntry::kValueMaxLength, 0);
   while (current_experiment < experiment_strings_size &&
          num_chunks < kMaxReportedExperimentChunks) {
@@ skipping 35 matching lines @@
       base::StringPrintf(
           L"%d", static_cast<int>(experiment_strings_size)).c_str(),
       google_breakpad::CustomInfoEntry::kValueMaxLength);
 }
 
 namespace testing {
 
 void SetExperimentList(const std::vector<string16>& experiment_strings) {
   std::vector<const wchar_t*> cstrings;
   StringVectorToCStringVector(experiment_strings, &cstrings);
-  ::SetExperimentList(&cstrings[0], cstrings.size());
+  ::SetExperimentList2(&cstrings[0], cstrings.size());
 }
 
 }  // namespace testing