base/version: remove wstring version

chrome/app/client_util.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 <windows.h>
 #include <shlwapi.h>
 
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
 #include "base/environment.h"
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/scoped_ptr.h"
+#include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "base/version.h"
 #include "chrome/app/breakpad_win.h"
 #include "chrome/app/client_util.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/result_codes.h"
 #include "chrome/installer/util/browser_distribution.h"
 #include "chrome/installer/util/install_util.h"
 #include "chrome/installer/util/google_update_constants.h"
@@ skipping 182 matching lines @@
 // If that fails then finally we look at the registry which should point us
 // to the latest version. This is the expected path for the first chrome.exe
 // browser instance in an installed build.
 HMODULE MainDllLoader::Load(std::wstring* out_version, std::wstring* out_file) {
   std::wstring dir(GetExecutablePath());
   *out_file = dir;
   HMODULE dll = LoadChromeWithDirectory(out_file);
   if (dll)
     return dll;
 
-  std::wstring version_env_string;
+  std::wstring version_string;
   scoped_ptr<Version> version;
   const CommandLine& cmd_line = *CommandLine::ForCurrentProcess();
   if (cmd_line.HasSwitch(switches::kChromeVersion)) {
-    version_env_string = cmd_line.GetSwitchValueNative(
-        switches::kChromeVersion);
-    version.reset(Version::GetVersionFromString(version_env_string));
+    version_string = cmd_line.GetSwitchValueNative(switches::kChromeVersion);
+    version.reset(Version::GetVersionFromString(WideToASCII(version_string)));
 
     if (!version.get()) {
       // If a bogus command line flag was given, then abort.
       LOG(ERROR) << "Invalid version string received on command line: "
-                 << version_env_string;
+                 << version_string;
       return NULL;
     }
   }
 
   if (!version.get()) {
     if (EnvQueryStr(ASCIIToWide(chrome::kChromeVersionEnvVar).c_str(),
-                    &version_env_string)) {
-      version.reset(Version::GetVersionFromString(version_env_string));
+                    &version_string)) {
+      version.reset(Version::GetVersionFromString(WideToASCII(version_string)));
     }
   }
 
   if (!version.get()) {
     std::wstring reg_path(GetRegistryPath());
     // Look into the registry to find the latest version. We don't validate
     // this by building a Version object to avoid harming normal case startup
     // time.
-    version_env_string.clear();
-    GetVersion(dir.c_str(), reg_path.c_str(), &version_env_string);
+    version_string.clear();
+    GetVersion(dir.c_str(), reg_path.c_str(), &version_string);
   }
 
-  if (version.get() || !version_env_string.empty()) {
+  if (version.get() || !version_string.empty()) {
     *out_file = dir;
-    *out_version = version_env_string;
+    *out_version = version_string;
     out_file->append(*out_version).append(L"\\");
     return LoadChromeWithDirectory(out_file);
   } else {
     return NULL;
   }
 }
 
 // Launching is a matter of loading the right dll, setting the CHROME_VERSION
 // environment variable and just calling the entry point. Derived classes can
 // add custom code in the OnBeforeLaunch callback.
@@ skipping 71 matching lines @@
   }
 };
 
 MainDllLoader* MakeMainDllLoader() {
 #if defined(GOOGLE_CHROME_BUILD)
   return new ChromeDllLoader();
 #else
   return new ChromiumDllLoader();
 #endif
 }