[cros-login-refactor] Move CompleteOffTheRecordLogin to UserSessionManager.

chrome/browser/chromeos/login/existing_user_controller_browsertest.cc

Index: chrome/browser/chromeos/login/existing_user_controller_browsertest.cc
diff --git a/chrome/browser/chromeos/login/existing_user_controller_browsertest.cc b/chrome/browser/chromeos/login/existing_user_controller_browsertest.cc
index c1341417a66f445445c67f3d64aea91252be78f7..af8078cf8c716f4e582a40095b225fbf0be82a8d 100644
--- a/chrome/browser/chromeos/login/existing_user_controller_browsertest.cc
+++ b/chrome/browser/chromeos/login/existing_user_controller_browsertest.cc
@@ -103,6 +103,9 @@ class ExistingUserControllerTest : public policy::DevicePolicyCrosBrowserTest {
     LoginUtils::Set(mock_login_utils_);
     EXPECT_CALL(*mock_login_utils_, DelegateDeleted(_))
         .Times(1);
+    EXPECT_CALL(*mock_login_utils_, RestartToApplyPerSessionFlagsIfNeed(_, _))

[ygorshenin1, 2014/10/13 12:20:50]

Why this change is needed?

[Nikita (slow), 2014/10/13 12:22:36]

This method call has been added a while ago and now adding this expectation just
to silence warning of non-defined method call.

I guess I'll actually remove it on one of the future iterations.

Just minimizing logs noise.

+        .Times(AnyNumber())
+        .WillRepeatedly(Return(false));
 
     mock_login_display_host_.reset(new MockLoginDisplayHost());
     mock_login_display_ = new MockLoginDisplay();
@@ -599,8 +602,6 @@ IN_PROC_BROWSER_TEST_F(ExistingUserControllerPublicSessionTest,
   EXPECT_CALL(*mock_login_utils_, CreateAuthenticator(_))
       .Times(1)
       .WillOnce(WithArg<0>(CreateAuthenticator(user_context)));
-  EXPECT_CALL(*mock_login_utils_, CompleteOffTheRecordLogin(_))
-      .Times(1);
 
   existing_user_controller()->OnSigninScreenReady();
   SetAutoLoginPolicy(kPublicSessionAccountId, kAutoLoginLongDelay);