With force signin enabled, the signin dialog will be displayed after profile creation instead of br…

chrome/browser/ui/webui/signin/signin_utils.cc

Index: chrome/browser/ui/webui/signin/signin_utils.cc
diff --git a/chrome/browser/ui/webui/signin/signin_utils.cc b/chrome/browser/ui/webui/signin/signin_utils.cc
index aa6711d4c2fb5d760b0b0dfac214222629a8a58b..c336310d8a6836bba887db2e52840b0f7dd80727 100644
--- a/chrome/browser/ui/webui/signin/signin_utils.cc
+++ b/chrome/browser/ui/webui/signin/signin_utils.cc
@@ -7,9 +7,12 @@
 #include <set>
 
 #include "base/bind.h"
+#include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser_finder.h"
+#include "chrome/common/pref_names.h"
 #include "components/guest_view/browser/guest_view_manager.h"
+#include "components/prefs/pref_service.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_ui.h"
 #include "extensions/browser/guest_view/web_view/web_view_guest.h"
@@ -75,4 +78,10 @@ void SetInitializedModalHeight(content::WebUI* web_ui,
   }
 }
 
+bool IsForceSigninEnabled() {
+  PrefService* prefs = g_browser_process->local_state();
+  DCHECK(prefs);

[anthonyvd, 2016/09/30 17:30:15]

This DCHECK seems superfluous since prefs is dereferenced on the next line.
Since not having a local_state here is probably an unrecoverable edge case (and
Chrome probably crashed before getting here) I'd probably just remove the DCHECK
altogether.

[zmin, 2016/09/30 17:41:48]

Done.

+  return prefs->GetBoolean(prefs::kForceBrowserSignin);
+}
+
 }  // namespace signin