Do not clear tokens from db when the user enters new credentials.

chrome/browser/sync/signin_manager_unittest.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/sync/signin_manager.h"
 
 #include "base/test/signaling_task.h"
 #include "chrome/browser/net/gaia/token_service.h"
 #include "chrome/browser/net/gaia/token_service_unittest.h"
 #include "chrome/browser/password_manager/encryptor.h"
+#include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/sync/util/oauth.h"
 #include "chrome/browser/webdata/web_data_service.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/net/gaia/gaia_urls.h"
+#include "chrome/common/pref_names.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/test/test_url_fetcher_factory.h"
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_status.h"
 
 #include "testing/gtest/include/gtest/gtest.h"
 
 class SigninManagerTest : public TokenServiceTestHarness {
  public:
   virtual void SetUp() OVERRIDE {
@@ skipping 80 matching lines @@
   // Should go into token service and stop.
   EXPECT_EQ(1U, google_login_success_.size());
   EXPECT_EQ(0U, google_login_failure_.size());
 
   // Should persist across resets.
   manager_.reset(new SigninManager());
   manager_->Initialize(profile_.get());
   EXPECT_EQ("user@gmail.com", manager_->GetUsername());
 }
 
+TEST_F(SigninManagerTest, ClearInMemoryData) {
+  browser_sync::SetIsUsingOAuthForTest(false);
+  manager_->Initialize(profile_.get());
+  EXPECT_TRUE(manager_->GetUsername().empty());
+
+  manager_->StartSignIn("username", "password", "", "");
+  EXPECT_FALSE(manager_->GetUsername().empty());
+
+  SimulateValidResponseClientLogin();
+
+  // Should go into token service and stop.
+  EXPECT_EQ(1U, google_login_success_.size());
+  EXPECT_EQ(0U, google_login_failure_.size());
+
+  EXPECT_EQ("user@gmail.com", manager_->GetUsername());
+
+  // Now clear the in memory data.
+  manager_->ClearInMemoryData();
+  EXPECT_TRUE(manager_->GetUsername().empty());
+
+  // Ensure preferences are not modified.
+  EXPECT_FALSE(
+     profile_->GetPrefs()->GetString(prefs::kGoogleServicesUsername).empty());
+
+  // On reset it should be regenerated.
+  manager_.reset(new SigninManager());
+  manager_->Initialize(profile_.get());
+
+  // Now make sure we have the right user name.
+  EXPECT_EQ("user@gmail.com", manager_->GetUsername());
+}
+
+
 // NOTE: OAuth's "StartOAuthSignIn" is called before collecting credentials
 //       from the user.  See also SigninManagerTest::SignInClientLogin.
 TEST_F(SigninManagerTest, SignInOAuth) {
   browser_sync::SetIsUsingOAuthForTest(true);
   manager_->Initialize(profile_.get());
   EXPECT_TRUE(manager_->GetUsername().empty());
 
   SimulateValidSigninOAuth();
   EXPECT_FALSE(manager_->GetUsername().empty());
 
@@ skipping 121 matching lines @@
 TEST_F(SigninManagerTest, SignOutOnUserInfoSucessRaceTest) {
   browser_sync::SetIsUsingOAuthForTest(true);
   manager_->Initialize(profile_.get());
   EXPECT_TRUE(manager_->GetUsername().empty());
 
   SimulateSigninStartOAuth();
   manager_->SignOut();
   SimulateOAuthUserInfoSuccess();
   EXPECT_TRUE(manager_->GetUsername().empty());
 }