diff options
author | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-03 03:00:50 +0000 |
---|---|---|
committer | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-03 03:00:50 +0000 |
commit | 76b90d310def447b7b5f10d92a69813308150778 (patch) | |
tree | 24c61fdccf94360913bfa8f7ac67c24a27bcfb38 /net/proxy/proxy_config_service_linux_unittest.cc | |
parent | 7f2a9dbe56bfa6189271af808c53ccaee193a961 (diff) | |
download | chromium_src-76b90d310def447b7b5f10d92a69813308150778.zip chromium_src-76b90d310def447b7b5f10d92a69813308150778.tar.gz chromium_src-76b90d310def447b7b5f10d92a69813308150778.tar.bz2 |
base: Rename EnvVarGetter to Environment.
Now EnvVarGetter do much more than getting environment variables.
Per suggestion from Pawel in http://codereview.chromium.org/3043018/.
BUG=None
TEST=trybots
Signed-off-by: Thiago Farina <tfarina@chromium.org>
Review URL: http://codereview.chromium.org/3052034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54696 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/proxy/proxy_config_service_linux_unittest.cc')
-rw-r--r-- | net/proxy/proxy_config_service_linux_unittest.cc | 70 |
1 files changed, 35 insertions, 35 deletions
diff --git a/net/proxy/proxy_config_service_linux_unittest.cc b/net/proxy/proxy_config_service_linux_unittest.cc index 30c3311..5a8f292 100644 --- a/net/proxy/proxy_config_service_linux_unittest.cc +++ b/net/proxy/proxy_config_service_linux_unittest.cc @@ -78,9 +78,9 @@ struct SettingsTable { map_type settings; }; -class MockEnvVarGetter : public base::EnvVarGetter { +class MockEnvironment : public base::Environment { public: - MockEnvVarGetter() { + MockEnvironment() { #define ENTRY(x) table.settings[#x] = &values.x ENTRY(DESKTOP_SESSION); ENTRY(HOME); @@ -605,10 +605,10 @@ TEST_F(ProxyConfigServiceLinuxTest, BasicGConfTest) { for (size_t i = 0; i < ARRAYSIZE_UNSAFE(tests); ++i) { SCOPED_TRACE(StringPrintf("Test[%" PRIuS "] %s", i, tests[i].description.c_str())); - MockEnvVarGetter* env_getter = new MockEnvVarGetter; + MockEnvironment* env = new MockEnvironment; MockGConfSettingGetter* gconf_getter = new MockGConfSettingGetter; SynchConfigGetter sync_config_getter( - new ProxyConfigServiceLinux(env_getter, gconf_getter)); + new ProxyConfigServiceLinux(env, gconf_getter)); ProxyConfig config; gconf_getter->values = tests[i].values; sync_config_getter.SetupAndInitialFetch(); @@ -895,12 +895,12 @@ TEST_F(ProxyConfigServiceLinuxTest, BasicEnvTest) { for (size_t i = 0; i < ARRAYSIZE_UNSAFE(tests); ++i) { SCOPED_TRACE(StringPrintf("Test[%" PRIuS "] %s", i, tests[i].description.c_str())); - MockEnvVarGetter* env_getter = new MockEnvVarGetter; + MockEnvironment* env = new MockEnvironment; MockGConfSettingGetter* gconf_getter = new MockGConfSettingGetter; SynchConfigGetter sync_config_getter( - new ProxyConfigServiceLinux(env_getter, gconf_getter)); + new ProxyConfigServiceLinux(env, gconf_getter)); ProxyConfig config; - env_getter->values = tests[i].values; + env->values = tests[i].values; sync_config_getter.SetupAndInitialFetch(); sync_config_getter.SyncGetLatestProxyConfig(&config); @@ -911,10 +911,10 @@ TEST_F(ProxyConfigServiceLinuxTest, BasicEnvTest) { } TEST_F(ProxyConfigServiceLinuxTest, GconfNotification) { - MockEnvVarGetter* env_getter = new MockEnvVarGetter; + MockEnvironment* env = new MockEnvironment; MockGConfSettingGetter* gconf_getter = new MockGConfSettingGetter; ProxyConfigServiceLinux* service = - new ProxyConfigServiceLinux(env_getter, gconf_getter); + new ProxyConfigServiceLinux(env, gconf_getter); SynchConfigGetter sync_config_getter(service); ProxyConfig config; @@ -1292,13 +1292,13 @@ TEST_F(ProxyConfigServiceLinuxTest, KDEConfigParser) { for (size_t i = 0; i < ARRAYSIZE_UNSAFE(tests); ++i) { SCOPED_TRACE(StringPrintf("Test[%" PRIuS "] %s", i, tests[i].description.c_str())); - MockEnvVarGetter* env_getter = new MockEnvVarGetter; - env_getter->values = tests[i].env_values; + MockEnvironment* env = new MockEnvironment; + env->values = tests[i].env_values; // Force the KDE getter to be used and tell it where the test is. - env_getter->values.DESKTOP_SESSION = "kde4"; - env_getter->values.KDEHOME = kde_home_.value().c_str(); + env->values.DESKTOP_SESSION = "kde4"; + env->values.KDEHOME = kde_home_.value().c_str(); SynchConfigGetter sync_config_getter( - new ProxyConfigServiceLinux(env_getter)); + new ProxyConfigServiceLinux(env)); ProxyConfig config; // Overwrite the kioslaverc file. file_util::WriteFile(kioslaverc_, tests[i].kioslaverc.c_str(), @@ -1328,11 +1328,11 @@ TEST_F(ProxyConfigServiceLinuxTest, KDEHomePicker) { CHECK(!file_util::DirectoryExists(kde4_home_)); { SCOPED_TRACE("KDE4, no .kde4 directory, verify fallback"); - MockEnvVarGetter* env_getter = new MockEnvVarGetter; - env_getter->values.DESKTOP_SESSION = "kde4"; - env_getter->values.HOME = user_home_.value().c_str(); + MockEnvironment* env = new MockEnvironment; + env->values.DESKTOP_SESSION = "kde4"; + env->values.HOME = user_home_.value().c_str(); SynchConfigGetter sync_config_getter( - new ProxyConfigServiceLinux(env_getter)); + new ProxyConfigServiceLinux(env)); ProxyConfig config; sync_config_getter.SetupAndInitialFetch(); sync_config_getter.SyncGetLatestProxyConfig(&config); @@ -1347,11 +1347,11 @@ TEST_F(ProxyConfigServiceLinuxTest, KDEHomePicker) { CHECK(file_util::PathExists(kioslaverc4_)); { SCOPED_TRACE("KDE4, .kde4 directory present, use it"); - MockEnvVarGetter* env_getter = new MockEnvVarGetter; - env_getter->values.DESKTOP_SESSION = "kde4"; - env_getter->values.HOME = user_home_.value().c_str(); + MockEnvironment* env = new MockEnvironment; + env->values.DESKTOP_SESSION = "kde4"; + env->values.HOME = user_home_.value().c_str(); SynchConfigGetter sync_config_getter( - new ProxyConfigServiceLinux(env_getter)); + new ProxyConfigServiceLinux(env)); ProxyConfig config; sync_config_getter.SetupAndInitialFetch(); sync_config_getter.SyncGetLatestProxyConfig(&config); @@ -1360,11 +1360,11 @@ TEST_F(ProxyConfigServiceLinuxTest, KDEHomePicker) { } { SCOPED_TRACE("KDE3, .kde4 directory present, ignore it"); - MockEnvVarGetter* env_getter = new MockEnvVarGetter; - env_getter->values.DESKTOP_SESSION = "kde"; - env_getter->values.HOME = user_home_.value().c_str(); + MockEnvironment* env = new MockEnvironment; + env->values.DESKTOP_SESSION = "kde"; + env->values.HOME = user_home_.value().c_str(); SynchConfigGetter sync_config_getter( - new ProxyConfigServiceLinux(env_getter)); + new ProxyConfigServiceLinux(env)); ProxyConfig config; sync_config_getter.SetupAndInitialFetch(); sync_config_getter.SyncGetLatestProxyConfig(&config); @@ -1373,12 +1373,12 @@ TEST_F(ProxyConfigServiceLinuxTest, KDEHomePicker) { } { SCOPED_TRACE("KDE4, .kde4 directory present, KDEHOME set to .kde"); - MockEnvVarGetter* env_getter = new MockEnvVarGetter; - env_getter->values.DESKTOP_SESSION = "kde4"; - env_getter->values.HOME = user_home_.value().c_str(); - env_getter->values.KDEHOME = kde_home_.value().c_str(); + MockEnvironment* env = new MockEnvironment; + env->values.DESKTOP_SESSION = "kde4"; + env->values.HOME = user_home_.value().c_str(); + env->values.KDEHOME = kde_home_.value().c_str(); SynchConfigGetter sync_config_getter( - new ProxyConfigServiceLinux(env_getter)); + new ProxyConfigServiceLinux(env)); ProxyConfig config; sync_config_getter.SetupAndInitialFetch(); sync_config_getter.SyncGetLatestProxyConfig(&config); @@ -1391,11 +1391,11 @@ TEST_F(ProxyConfigServiceLinuxTest, KDEHomePicker) { file_util::SetLastModifiedTime(kde4_config_, base::Time()); { SCOPED_TRACE("KDE4, very old .kde4 directory present, use .kde"); - MockEnvVarGetter* env_getter = new MockEnvVarGetter; - env_getter->values.DESKTOP_SESSION = "kde4"; - env_getter->values.HOME = user_home_.value().c_str(); + MockEnvironment* env = new MockEnvironment; + env->values.DESKTOP_SESSION = "kde4"; + env->values.HOME = user_home_.value().c_str(); SynchConfigGetter sync_config_getter( - new ProxyConfigServiceLinux(env_getter)); + new ProxyConfigServiceLinux(env)); ProxyConfig config; sync_config_getter.SetupAndInitialFetch(); sync_config_getter.SyncGetLatestProxyConfig(&config); |