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

chrome/browser/shell_integration_unittest.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/shell_integration.h"
 
 #include <map>
 
 #include "base/file_path.h"
 #include "base/file_util.h"
@@ skipping 30 matching lines @@
 
   virtual bool GetEnv(const char* variable_name, std::string* result) {
     if (ContainsKey(variables_, variable_name)) {
       *result = variables_[variable_name];
       return true;
     }
 
     return false;
   }
 
+  virtual void SetEnv(const char* variable_name, const std::string& new_value) {
+    NOTIMPLEMENTED();
+  }
+
  private:
   std::map<std::string, std::string> variables_;
 
   DISALLOW_COPY_AND_ASSIGN(MockEnvVarGetter);
 };
 
 }  // namespace
 
 TEST(ShellIntegrationTest, GetDesktopShortcutTemplate) {
 #if defined(GOOGLE_CHROME_BUILD)
@@ skipping 240 matching lines @@
   // Non-default profile path should get chrome::kBrowserAppID joined with
   // profile info.
   FilePath profile_path(FILE_PATH_LITERAL("root"));
   profile_path = profile_path.Append(FILE_PATH_LITERAL("udd"));
   profile_path = profile_path.Append(FILE_PATH_LITERAL("User Data - Test"));
   EXPECT_EQ(BrowserDistribution::GetDistribution()->GetBrowserAppId() +
             L".udd.UserDataTest",
             ShellIntegration::GetChromiumAppId(profile_path));
 }
 #endif