OAuth2 sign-in flow for ChromeOS

chrome/browser/chromeos/login/login_performer.cc

Index: chrome/browser/chromeos/login/login_performer.cc
diff --git a/chrome/browser/chromeos/login/login_performer.cc b/chrome/browser/chromeos/login/login_performer.cc
index ca0a1ac9ffba64ef3e4cb1e1fcbc59bb4bdaad52..5a03b6778536eead3f60f7340826f40d918ce177 100644
--- a/chrome/browser/chromeos/login/login_performer.cc
+++ b/chrome/browser/chromeos/login/login_performer.cc
@@ -60,9 +60,6 @@ LoginPerformer::LoginPerformer(Delegate* delegate)
       screen_lock_requested_(false),
       initial_online_auth_pending_(false),
       auth_mode_(AUTH_MODE_INTERNAL),
-      using_oauth_(
-          !CommandLine::ForCurrentProcess()->HasSwitch(
-              switches::kSkipOAuthLogin)),
       ALLOW_THIS_IN_INITIALIZER_LIST(weak_factory_(this)) {
   DCHECK(default_performer_ == NULL)
       << "LoginPerformer should have only one instance.";
@@ -190,8 +187,8 @@ void LoginPerformer::OnProfileCreated(
       return;
   }
 
-  if (using_oauth_)
-    LoginUtils::Get()->StartTokenServices(profile);
+  // TODO(zelidrag): Figure out when this needs to run.
+  LoginUtils::Get()->StartTokenServices(profile);

[Joao da Silva, 2013/01/11 16:45:07]

AFAICT this method is dead code.

It's only used as the callback for CreateDefaultProfileAsync above. However,
that is only posted when |delegate_| is NULL, which I think that never happens.

LoginPerformer is only created by existing_user_controller.cc, and it always
sets the ExistingUserController as the delegate (except for tests). It only
removes the delegate at ExistingUserController::OnLoginSuccess(), but it deletes
the LoginPerformer immediately after.

In any case, StartTokenServices is currently only starting the fetch of the
oauth token for the policy server. This is usually handled by
LoginUtils::PrepareProfile() or LoginUtils::OnProfileCreated(). If there is no
|delegate_| here then this should be invoked here; but AFAICT this method is
never used.

[zel, 2013/01/11 19:51:16]

I suspect this might have something to do with screen locker then. Nikita might
know better. I will be happy to nuke it if I can't find when its being used.

[Nikita (slow), 2013/01/11 22:02:03]

Yes, this method could be removed.

[zel, 2013/01/12 02:07:36]

Removed LoginPerformer::OnProfileCreated(), branch of delegate_==NULL in
LoginPerformer::OnLoginSuccess() and removed LoginUtils::StartTokenServices().

 
   // Don't unlock screen if it was locked while we're waiting
   // for initial online auth.