ChromeOS NetworkScreenHandler should not call CheckAndResolveLocale on UI thread.

chrome/browser/chromeos/base/locale_util.h

Index: chrome/browser/chromeos/base/locale_util.h
diff --git a/chrome/browser/chromeos/base/locale_util.h b/chrome/browser/chromeos/base/locale_util.h
index cf252a470c622c660f63d45890cee5bf7025ecb8..e9b517fe298305234efb36ab8105bdf7cc9b284d 100644
--- a/chrome/browser/chromeos/base/locale_util.h
+++ b/chrome/browser/chromeos/base/locale_util.h
@@ -21,6 +21,17 @@ class Callback;
 namespace chromeos {
 namespace locale_util {
 
+class LanguageSwitchResult {
+ public:
+  LanguageSwitchResult(const std::string& requested_locale,
+                       const std::string& loaded_locale,
+                       bool success);
+
+  std::string requested_locale;
+  std::string loaded_locale;
+  bool success;
+};
+
 // This callback is called on UI thread, when ReloadLocaleResources() is
 // completed on BlockingPool.
 // Arguments:

[dzhioev (left Google), 2014/10/02 14:12:10]

Update comment.

[Alexander Alekseev, 2014/10/02 19:05:03]

Done.

@@ -28,7 +39,7 @@ namespace locale_util {
 //   loaded_locale - actual locale name loaded.
 //   success - if locale load succeeded.
 // (const std::string* locale, const std::string* loaded_locale, bool success)
-typedef base::Callback<void(const std::string&, const std::string&, bool)>
+typedef base::Callback<void(scoped_ptr<LanguageSwitchResult> result)>

[dzhioev (left Google), 2014/10/02 14:12:10]

I don't think SwitchLanguageCallback should receive scoped_ptr, rather than
const ref.  

[Alexander Alekseev, 2014/10/02 19:05:03]

Done.

     SwitchLanguageCallback;
 
 // This function updates input methods only if requested. In general, you want