diff options
author | benwells@chromium.org <benwells@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-16 06:32:59 +0000 |
---|---|---|
committer | benwells@chromium.org <benwells@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-16 06:32:59 +0000 |
commit | e0879c19fe65c1455d6eab36a4a01524465ffb08 (patch) | |
tree | df3400ef596ecf62cc44bbf7687417c2dcbabf3a /rlz/chromeos | |
parent | cee6a92d0d15ba27677d1f51c0a03d8990832445 (diff) | |
download | chromium_src-e0879c19fe65c1455d6eab36a4a01524465ffb08.zip chromium_src-e0879c19fe65c1455d6eab36a4a01524465ffb08.tar.gz chromium_src-e0879c19fe65c1455d6eab36a4a01524465ffb08.tar.bz2 |
Revert 270872 "Move all callers of GetHomeDir() to PathService::..."
> Move all callers of GetHomeDir() to PathService::Get(base::DIR_HOME).
>
> * Fixes GetHomeDir() for multi-profiles case on Chrome OS.
> * Once user signs in on Chrome OS base::DIR_HOME is overridden with primary user homedir.
> * Added content switch --homedir to pass that information to ppapi plugins since they run in a separate process and previous base::DIR_HOME override does not apply there.
>
> This fix doesn't require checking for --multi-profiles switch
> since user_id hash is known even without it.
>
> BUG=331530
> TBR=vitalybuka@chromium.org
>
> Review URL: https://codereview.chromium.org/200473002
This test has caused two tests to start failing on the chromeos valgrind bots:
http://build.chromium.org/p/chromium.memory.fyi/builders/Chromium%20OS%20%28valgrind%29%286%29/builds/25884
http://build.chromium.org/p/chromium.memory.fyi/builders/Chromium%20OS%20%28valgrind%29%285%29/builds/26990
TBR=nkostylev@chromium.org
Review URL: https://codereview.chromium.org/284333002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270951 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'rlz/chromeos')
-rw-r--r-- | rlz/chromeos/lib/rlz_value_store_chromeos.cc | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/rlz/chromeos/lib/rlz_value_store_chromeos.cc b/rlz/chromeos/lib/rlz_value_store_chromeos.cc index c54a194..0cd87af 100644 --- a/rlz/chromeos/lib/rlz_value_store_chromeos.cc +++ b/rlz/chromeos/lib/rlz_value_store_chromeos.cc @@ -4,13 +4,11 @@ #include "rlz/chromeos/lib/rlz_value_store_chromeos.h" -#include "base/base_paths.h" #include "base/file_util.h" #include "base/files/important_file_writer.h" #include "base/json/json_file_value_serializer.h" #include "base/json/json_string_value_serializer.h" #include "base/logging.h" -#include "base/path_service.h" #include "base/sequenced_task_runner.h" #include "base/strings/string_number_conversions.h" #include "base/values.h" @@ -44,19 +42,15 @@ base::FilePath g_testing_rlz_store_path_; // Returns file path of the RLZ storage. base::FilePath GetRlzStorePath() { - base::FilePath homedir; - PathService::Get(base::DIR_HOME, &homedir); return g_testing_rlz_store_path_.empty() ? - homedir.Append(kRLZDataFileName) : + base::GetHomeDir().Append(kRLZDataFileName) : g_testing_rlz_store_path_.Append(kRLZDataFileName); } // Returns file path of the RLZ storage lock file. base::FilePath GetRlzStoreLockPath() { - base::FilePath homedir; - PathService::Get(base::DIR_HOME, &homedir); return g_testing_rlz_store_path_.empty() ? - homedir.Append(kRLZLockFileName) : + base::GetHomeDir().Append(kRLZLockFileName) : g_testing_rlz_store_path_.Append(kRLZLockFileName); } |