| Index: chrome/browser/signin/signin_manager_unittest.cc
|
| diff --git a/chrome/browser/signin/signin_manager_unittest.cc b/chrome/browser/signin/signin_manager_unittest.cc
|
| index f008430cf37a7db7b9b322562cc2e717e29db8d2..160a03ae47cc106f3ed2d1d5efedf0e0c45f1143 100644
|
| --- a/chrome/browser/signin/signin_manager_unittest.cc
|
| +++ b/chrome/browser/signin/signin_manager_unittest.cc
|
| @@ -23,7 +23,9 @@
|
| #include "chrome/test/base/testing_profile.h"
|
| #include "components/webdata/encryptor/encryptor.h"
|
| #include "content/public/browser/child_process_security_policy.h"
|
| +#include "content/public/browser/storage_partition.h"
|
| #include "content/public/test/test_browser_thread_bundle.h"
|
| +#include "content/public/test/test_utils.h"
|
| #include "google_apis/gaia/gaia_constants.h"
|
| #include "google_apis/gaia/gaia_urls.h"
|
| #include "net/cookies/cookie_monster.h"
|
| @@ -200,6 +202,12 @@ class SigninManagerTest : public TokenServiceTestHarness {
|
| manager_->SignOut();
|
| }
|
|
|
| + net::CookieMonster* GetCookieMonster(const GURL& origin) {
|
| + // Since it's a unittest, assume default StoragePartition.
|
| + return content::BrowserContext::GetDefaultStoragePartition(profile())->
|
| + GetCookieStoreForScheme(origin.scheme())->GetCookieMonster();
|
| + }
|
| +
|
| net::TestURLFetcherFactory factory_;
|
| scoped_ptr<SigninManager> manager_;
|
| content::TestNotificationTracker google_login_success_;
|
| @@ -289,14 +297,14 @@ TEST_F(SigninManagerTest, SignInWithCredentialsEmptyPasswordValidCookie) {
|
| EXPECT_TRUE(manager_->GetAuthenticatedUsername().empty());
|
|
|
| // Set a valid LSID cookie in the test cookie store.
|
| - scoped_refptr<net::CookieMonster> cookie_monster =
|
| - profile()->GetCookieMonster();
|
| + GURL origin("https://accounts.google.com");
|
| + scoped_refptr<net::CookieMonster> cookie_monster = GetCookieMonster(origin);
|
| net::CookieOptions options;
|
| options.set_include_httponly();
|
| cookie_monster->SetCookieWithOptionsAsync(
|
| - GURL("https://accounts.google.com"),
|
| - "LSID=1234; secure; httponly", options,
|
| - net::CookieMonster::SetCookiesCallback());
|
| + origin,
|
| + "LSID=1234; secure; httponly", options,
|
| + net::CookieMonster::SetCookiesCallback());
|
|
|
| // Since the password is empty, will verify the gaia cookies first.
|
| manager_->StartSignInWithCredentials(
|
| @@ -334,14 +342,14 @@ TEST_F(SigninManagerTest, SignInWithCredentialsEmptyPasswordInValidCookie) {
|
| EXPECT_TRUE(manager_->GetAuthenticatedUsername().empty());
|
|
|
| // Set an invalid LSID cookie in the test cookie store.
|
| - scoped_refptr<net::CookieMonster> cookie_monster =
|
| - profile()->GetCookieMonster();
|
| + GURL origin("https://accounts.google.com");
|
| + scoped_refptr<net::CookieMonster> cookie_monster = GetCookieMonster(origin);
|
| net::CookieOptions options;
|
| options.set_include_httponly();
|
| cookie_monster->SetCookieWithOptionsAsync(
|
| - GURL("https://accounts.google.com"),
|
| - "LSID=1234; domain=google.com; secure; httponly", options,
|
| - net::CookieMonster::SetCookiesCallback());
|
| + origin,
|
| + "LSID=1234; domain=google.com; secure; httponly", options,
|
| + net::CookieMonster::SetCookiesCallback());
|
|
|
| // Since the password is empty, must verify the gaia cookies first.
|
| manager_->StartSignInWithCredentials(
|
|
|