Display local signin error without browser and record the path of selected profile in user manager.

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

 // Copyright 2016 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/ui/webui/signin/signin_error_ui.h"
 
+#include <vector>
+
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile_attributes_entry.h"
 #include "chrome/browser/profiles/profile_attributes_storage.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/browser_window.h"
+#include "chrome/browser/ui/user_manager.h"
 #include "chrome/browser/ui/webui/signin/login_ui_service.h"
 #include "chrome/browser/ui/webui/signin/login_ui_service_factory.h"
 #include "chrome/browser/ui/webui/signin/signin_error_handler.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/grit/browser_resources.h"
 #include "chrome/grit/generated_resources.h"
 #include "content/public/browser/web_ui.h"
 #include "content/public/browser/web_ui_data_source.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/webui/web_ui_util.h"
 #include "ui/gfx/text_elider.h"
 
+namespace {
+bool IsSystemProfile(Profile* profile) {
+  return profile->GetOriginalProfile()->IsSystemProfile();
+}
+}  // namespace

[tommycli, 2016/09/21 18:26:59]

nit: I don't think this shortening is worth it. It's only used in two places.

[zmin, 2016/09/21 21:21:37]

Why not? That is still a code copy.

[tommycli, 2016/09/21 21:26:16]

I think since it's so short, it would be easier to read to just inline
profile->GetOriginalProfile()->IsSystemProfile().

[zmin, 2016/09/21 21:52:05]

Done.

+
 SigninErrorUI::SigninErrorUI(content::WebUI* web_ui)
-    : SigninErrorUI(web_ui, new SigninErrorHandler) {}
+    : SigninErrorUI(web_ui,
+                    SigninErrorHandler::Create(
+                        IsSystemProfile(Profile::FromWebUI(web_ui)))) {}
 
 SigninErrorUI::SigninErrorUI(content::WebUI* web_ui,
                              SigninErrorHandler* handler)
     : WebDialogUI(web_ui) {
-  Profile* profile = Profile::FromWebUI(web_ui);
+  Profile* webui_profile = Profile::FromWebUI(web_ui);
+  Profile* signin_profile;
+  bool is_system_profile = IsSystemProfile(webui_profile);
+  if (is_system_profile) {

[tommycli, 2016/09/21 18:26:59]

nit: Maybe clearer with a ternary expression? Not required, just a maybe
suggestion.

[zmin, 2016/09/21 21:21:37]

I'm personally not a fan of ternary expression. Especially when the expression
is longer than one line.

[tommycli, 2016/09/21 21:26:15]

Acknowledged.

+    signin_profile = g_browser_process->profile_manager()->GetProfileByPath(
+        UserManager::GetSigninProfilePath());
+  } else {
+    signin_profile = webui_profile;
+  }
+
   content::WebUIDataSource* source =
       content::WebUIDataSource::Create(chrome::kChromeUISigninErrorHost);
   source->SetJsonPath("strings.js");
   source->SetDefaultResource(IDR_SIGNIN_ERROR_HTML);
   source->AddResourcePath("signin_error.js", IDR_SIGNIN_ERROR_JS);
   source->AddResourcePath("signin_shared_css.html", IDR_SIGNIN_SHARED_CSS_HTML);
+  source->AddBoolean("isSystemProfile", is_system_profile);
 
   // Retrieve the last signin error message and email used.
   LoginUIService* login_ui_service =
-      LoginUIServiceFactory::GetForProfile(profile);
+      LoginUIServiceFactory::GetForProfile(signin_profile);
   const base::string16 last_login_result(
       login_ui_service->GetLastLoginResult());
   const base::string16 email = login_ui_service->GetLastLoginErrorEmail();
   if (email.empty()) {
     source->AddLocalizedString("signinErrorTitle", IDS_SIGNIN_ERROR_TITLE);
   } else {
     source->AddString(
         "signinErrorTitle",
         l10n_util::GetStringFUTF16(IDS_SIGNIN_ERROR_EMAIL_TITLE, email));
   }
 
   // Tweak the dialog UI depending on whether the signin error is
-  // username-in-use error.
+  // username-in-use error and the error UI is shown with a browser window.
   base::string16 existing_name;
-  if (last_login_result.compare(
+  if (!is_system_profile &&
+      last_login_result.compare(
           l10n_util::GetStringUTF16(IDS_SYNC_USER_NAME_IN_USE_ERROR)) == 0) {
     ProfileManager* profile_manager = g_browser_process->profile_manager();
     if (profile_manager) {
       std::vector<ProfileAttributesEntry*> entries =
           profile_manager->GetProfileAttributesStorage()
               .GetAllProfilesAttributes();
       DCHECK(!email.empty());
       for (const ProfileAttributesEntry* entry : entries) {
         if (gaia::AreEmailsSame(base::UTF16ToUTF8(email),
                                 base::UTF16ToUTF8(entry->GetUserName()))) {
@@ skipping 25 matching lines @@
   source->AddLocalizedString("signinErrorCloseLabel",
                              IDS_SIGNIN_ERROR_CLOSE_BUTTON_LABEL);
   source->AddLocalizedString("signinErrorOkLabel",
                              IDS_SIGNIN_ERROR_OK_BUTTON_LABEL);
 
   base::DictionaryValue strings;
   webui::SetLoadTimeDataDefaults(g_browser_process->GetApplicationLocale(),
                                  &strings);
   source->AddLocalizedStrings(strings);
 
-  content::WebUIDataSource::Add(profile, source);
+  content::WebUIDataSource::Add(webui_profile, source);
   web_ui->AddMessageHandler(handler);
 }