base: Add SetEnv() to EnvVarGetter class and get rid of the some ifdefs.

chrome/common/logging_chrome_uitest.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 "build/build_config.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #endif
 
@@ skipping 11 matching lines @@
 
 class ChromeLoggingTest : public testing::Test {
  public:
   // Stores the current value of the log file name environment
   // variable and sets the variable to new_value.
   void SaveEnvironmentVariable(std::string new_value) {
     scoped_ptr<base::EnvVarGetter> env(base::EnvVarGetter::Create());
     if (!env->GetEnv(env_vars::kLogFileName, &environment_filename_))
       environment_filename_ = "";
 
-    // TODO(port) Add base::SetEnv() and get rid of the ifdefs.
-#if defined(OS_WIN)
-    SetEnvironmentVariable(ASCIIToWide(env_vars::kLogFileName).c_str(),
-                           ASCIIToWide(new_value).c_str());
-#else
-    setenv(env_vars::kLogFileName, new_value.c_str(), 1);
-#endif
+    env->SetEnv(env_vars::kLogFileName, new_value);
   }
 
   // Restores the value of the log file nave environment variable
   // previously saved by SaveEnvironmentVariable().
   void RestoreEnvironmentVariable() {
-#if defined(OS_WIN)
-    SetEnvironmentVariable(ASCIIToWide(env_vars::kLogFileName).c_str(),
-                           ASCIIToWide(environment_filename_).c_str());
-#else
-    setenv(env_vars::kLogFileName, environment_filename_.c_str(), 1);
-#endif
+    scoped_ptr<base::EnvVarGetter> env(base::EnvVarGetter::Create());
+    env->SetEnv(env_vars::kLogFileName, environment_filename_);
   }
 
  private:
   std::string environment_filename_;  // Saves real environment value.
 };
 
 // Tests the log file name getter without an environment variable.
 TEST_F(ChromeLoggingTest, LogFileName) {
   SaveEnvironmentVariable("");
 
@@ skipping 127 matching lines @@
   if (UITest::in_process_renderer()) {
     // in process mode doesn't do the crashing.
     expected_crashes_ = 0;
   } else {
     scoped_refptr<BrowserProxy> browser(automation()->GetBrowserWindow(0));
     ASSERT_TRUE(browser.get());
     ASSERT_TRUE(browser->WaitForTabCountToBecome(1, action_max_timeout_ms()));
     expected_crashes_ = EXPECTED_CRASH_CRASHES;
   }
 }