diff options
| author | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-16 03:34:25 +0000 |
|---|---|---|
| committer | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-16 03:34:25 +0000 |
| commit | e9032c6a2ea03e4a5081e0234eff857cf190e3e4 (patch) | |
| tree | a9e6ec385829730bc9442579c2fef525d1caf71e /base/xdg_util_unittest.cc | |
| parent | 3e108342206210d64d81368044d9503445a1a1f3 (diff) | |
| download | chromium_src-e9032c6a2ea03e4a5081e0234eff857cf190e3e4.zip chromium_src-e9032c6a2ea03e4a5081e0234eff857cf190e3e4.tar.gz chromium_src-e9032c6a2ea03e4a5081e0234eff857cf190e3e4.tar.bz2 | |
base: Return bool instead of void in EnvVarGetter::SetEnv.
Returning bool instead of void in this function is better, because with that
we can know if we had succeeded when tried to set an environment variable or not.
Also it makes this function more testable.
BUG=None
TEST=out/Debug/base_unittests --gtest_filter=EnvVarTest.*
Signed-off-by: Thiago Farina <tfarina@chromium.org>
Review URL: http://codereview.chromium.org/2891024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52602 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/xdg_util_unittest.cc')
| -rw-r--r-- | base/xdg_util_unittest.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/base/xdg_util_unittest.cc b/base/xdg_util_unittest.cc index 813242b..b821921 100644 --- a/base/xdg_util_unittest.cc +++ b/base/xdg_util_unittest.cc @@ -19,7 +19,7 @@ namespace { class MockEnvVarGetter : public base::EnvVarGetter { public: MOCK_METHOD2(GetEnv, bool(const char*, std::string* result)); - MOCK_METHOD2(SetEnv, void(const char*, const std::string& new_value)); + MOCK_METHOD2(SetEnv, bool(const char*, const std::string& new_value)); }; const char* kGnome = "gnome"; |