diff options
author | ajwong@chromium.org <ajwong@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-27 21:33:49 +0000 |
---|---|---|
committer | ajwong@chromium.org <ajwong@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-27 21:33:49 +0000 |
commit | 1c085a63a3d62c0bf5043a8336dce0e9923917f8 (patch) | |
tree | 3909ce2a3a51ec2f5173d63c97952eed0a128313 /remoting/client/client_util_unittest.cc | |
parent | 9e7fbfea4a79dbb7f3aa8e71f41b3ca4e40fe3b7 (diff) | |
download | chromium_src-1c085a63a3d62c0bf5043a8336dce0e9923917f8.zip chromium_src-1c085a63a3d62c0bf5043a8336dce0e9923917f8.tar.gz chromium_src-1c085a63a3d62c0bf5043a8336dce0e9923917f8.tar.bz2 |
Change ClientConfig to be a struct since it's just a dumb data carrier.
Also, the old method of passing around a pointer was not safe due to
object lifetime issues.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53851 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'remoting/client/client_util_unittest.cc')
-rw-r--r-- | remoting/client/client_util_unittest.cc | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/remoting/client/client_util_unittest.cc b/remoting/client/client_util_unittest.cc index d9f2e7c..4277769 100644 --- a/remoting/client/client_util_unittest.cc +++ b/remoting/client/client_util_unittest.cc @@ -25,12 +25,11 @@ TEST_F(ClientUtilTest, GetLoginInfoFromUrlParams) { std::string host_jid; ClientConfig config; - ASSERT_TRUE( - GetLoginInfoFromUrlParams(url, &config)); + ASSERT_TRUE(GetLoginInfoFromUrlParams(url, &config)); - EXPECT_EQ("auser", config.username()); - EXPECT_EQ("someauth", config.auth_token()); - EXPECT_EQ("ajid", config.host_jid()); + EXPECT_EQ("auser", config.username); + EXPECT_EQ("someauth", config.auth_token); + EXPECT_EQ("ajid", config.host_jid); } } // namespace remoting |