Add the reverse test to EnvironmentTest.

Add the reverse test to EnvironmentTest.

This test basically test another behavior of the GetVar function.
It tries to get a variable by looking for its reverse form.

BUG=None
TEST=out/Debug/base_unittests --gtest_filter=EnvironmentTest.GetVarReverse.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=56950

[Lei Zhang, 2010-08-20 03:55:16]

http://codereview.chromium.org/3162025/diff/1/2
File base/environment_unittest.cc (right):

http://codereview.chromium.org/3162025/diff/1/2#newcode32
base/environment_unittest.cc:32: EXPECT_NE(env_value, "");
Do you need both of these tests? Ditto on line 44.

[tfarina, 2010-08-20 04:01:47]

http://codereview.chromium.org/3162025/diff/1/2
File base/environment_unittest.cc (right):

http://codereview.chromium.org/3162025/diff/1/2#newcode32
base/environment_unittest.cc:32: EXPECT_NE(env_value, "");
On 2010/08/20 03:55:16, Lei Zhang wrote:
> Do you need both of these tests? Ditto on line 44.

They are not really necessary, since STREQ already check what we want. I will
remove these two checks.

[tfarina, 2010-08-20 04:01:47]

http://codereview.chromium.org/3162025/diff/1/2
File base/environment_unittest.cc (right):

http://codereview.chromium.org/3162025/diff/1/2#newcode32
base/environment_unittest.cc:32: EXPECT_NE(env_value, "");
On 2010/08/20 03:55:16, Lei Zhang wrote:
> Do you need both of these tests? Ditto on line 44.

They are not really necessary, since STREQ already check what we want. I will
remove these two checks.

[Lei Zhang, 2010-08-20 04:49:11]

LGTM!