Lock profile before sign in when force sign in is enabled.

chrome/browser/signin/chrome_signin_client.cc

 // Copyright 2014 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/signin/chrome_signin_client.h"
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 436 matching lines @@
 }
 
 void ChromeSigninClient::LockProfile(const base::FilePath& profile_path) {
   ProfileAttributesEntry* entry;
   bool has_entry =
       g_browser_process->profile_manager()
           ->GetProfileAttributesStorage()
           .GetProfileAttributesWithPath(profile_->GetPath(), &entry);
   if (!has_entry)
     return;
-  entry->SetIsSigninRequired(true);
+  entry->LockForceSigninProfile(true);

[anthonyvd, 2016/11/04 21:31:02]

Why isn't this still just SetIsSigninRequired considering that function calls
LockForceSigninProfile?  Doesn't this break non-force-signin lock?

[zmin, 2016/11/04 22:11:53]

LockProfile() is a function I added recently and will only be called for
force-sign-in sign out.

Yes, I agree that the function name confuses people after this change. So I'll
change that.

 }
 
 void ChromeSigninClient::ShowUserManager(const base::FilePath& profile_path) {
 #if !defined(OS_ANDROID) && !defined(OS_CHROMEOS)
   UserManager::Show(profile_path, profiles::USER_MANAGER_NO_TUTORIAL,
                     profiles::USER_MANAGER_SELECT_PROFILE_NO_ACTION);
 #endif
 }