Cleanup: Consolidate declarations of XDG variables.

chrome/common/chrome_paths_linux.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/common/chrome_paths_internal.h"
 
 #include "base/environment.h"
 #include "base/file_util.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/nix/xdg_util.h"
 #include "base/path_service.h"
 
 namespace {
 
-const char kDotConfigDir[] = ".config";
 const char kDownloadsDir[] = "Downloads";
 const char kPicturesDir[] = "Pictures";
-const char kXdgConfigHomeEnvVar[] = "XDG_CONFIG_HOME";
 
 }  // namespace
 
 namespace chrome {
 
+using base::nix::GetXDGDirectory;
+using base::nix::GetXDGUserDirectory;
+using base::nix::kDotConfigDir;
+using base::nix::kXdgConfigHomeEnvVar;
+
 // See http://standards.freedesktop.org/basedir-spec/basedir-spec-latest.html
 // for a spec on where config files go.  The net effect for most
 // systems is we use ~/.config/chromium/ for Chromium and
 // ~/.config/google-chrome/ for official builds.
 // (This also helps us sidestep issues with other apps grabbing ~/.chromium .)
 bool GetDefaultUserDataDirectory(FilePath* result) {
   scoped_ptr<base::Environment> env(base::Environment::Create());
-  FilePath config_dir(base::nix::GetXDGDirectory(env.get(),
-                                                 kXdgConfigHomeEnvVar,
-                                                 kDotConfigDir));
+  FilePath config_dir(GetXDGDirectory(env.get(),
+                                      kXdgConfigHomeEnvVar,
+                                      kDotConfigDir));
 #if defined(GOOGLE_CHROME_BUILD)
   *result = config_dir.Append("google-chrome");
 #else
   *result = config_dir.Append("chromium");
 #endif
   return true;
 }
 
 void GetUserCacheDirectory(const FilePath& profile_dir, FilePath* result) {
   // See http://standards.freedesktop.org/basedir-spec/basedir-spec-latest.html
   // for a spec on where cache files go.  Our rule is:
   // - if the user-data-dir in the standard place,
   //     use same subdirectory of the cache directory.
   //     (this maps ~/.config/google-chrome to ~/.cache/google-chrome as well
   //      as the same thing for ~/.config/chromium)
   // - otherwise, use the profile dir directly.
 
   // Default value in cases where any of the following fails.
   *result = profile_dir;
 
   scoped_ptr<base::Environment> env(base::Environment::Create());
 
   FilePath cache_dir;
   if (!PathService::Get(base::DIR_CACHE, &cache_dir))
     return;
-  FilePath config_dir(base::nix::GetXDGDirectory(env.get(),
-                                                 kXdgConfigHomeEnvVar,
-                                                 kDotConfigDir));
+  FilePath config_dir(GetXDGDirectory(env.get(),
+                                      kXdgConfigHomeEnvVar,
+                                      kDotConfigDir));
 
   if (!config_dir.AppendRelativePath(profile_dir, &cache_dir))
     return;
 
   *result = cache_dir;
 }
 
 bool GetChromeFrameUserDataDirectory(FilePath* result) {
   scoped_ptr<base::Environment> env(base::Environment::Create());
-  FilePath config_dir(base::nix::GetXDGDirectory(env.get(),
-                                                 kXdgConfigHomeEnvVar,
-                                                 kDotConfigDir));
+  FilePath config_dir(GetXDGDirectory(env.get(),
+                                      kXdgConfigHomeEnvVar,
+                                      kDotConfigDir));
 #if defined(GOOGLE_CHROME_BUILD)
   *result = config_dir.Append("google-chrome-frame");
 #else
   *result = config_dir.Append("chrome-frame");
 #endif
   return true;
 }
 
 bool GetUserDocumentsDirectory(FilePath* result) {
-  *result = base::nix::GetXDGUserDirectory("DOCUMENTS", "Documents");
+  *result = GetXDGUserDirectory("DOCUMENTS", "Documents");
   return true;
 }
 
 bool GetUserDownloadsDirectorySafe(FilePath* result) {
   FilePath home = file_util::GetHomeDir();
   *result = home.Append(kDownloadsDir);
   return true;
 }
 
 bool GetUserDownloadsDirectory(FilePath* result) {
   *result = base::nix::GetXDGUserDirectory("DOWNLOAD", kDownloadsDir);
   return true;
 }
 
 // We respect the user's preferred pictures location, unless it is
 // ~ or their desktop directory, in which case we default to ~/Pictures.
 bool GetUserPicturesDirectory(FilePath* result) {
-  *result = base::nix::GetXDGUserDirectory("PICTURES", kPicturesDir);
+  *result = GetXDGUserDirectory("PICTURES", kPicturesDir);
 
   FilePath home = file_util::GetHomeDir();
   if (*result != home) {
     FilePath desktop;
     GetUserDesktop(&desktop);
     if (*result != desktop) {
       return true;
     }
   }
 
   *result = home.Append(kPicturesDir);
   return true;
 }
 
 bool GetUserDesktop(FilePath* result) {
-  *result = base::nix::GetXDGUserDirectory("DESKTOP", "Desktop");
+  *result = GetXDGUserDirectory("DESKTOP", "Desktop");
   return true;
 }
 
 bool ProcessNeedsProfileDir(const std::string& process_type) {
   // For now we have no reason to forbid this on Linux as we don't
   // have the roaming profile troubles there. Moreover the Linux breakpad needs
   // profile dir access in all process if enabled on Linux.
   return true;
 }
 
 }  // namespace chrome