Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(133)

Unified Diff: chrome/browser/chromeos/login/login_utils.cc

Issue 620563002: ChromeOS NetworkScreenHandler should not call CheckAndResolveLocale on UI thread. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Cleanup. Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/login/login_utils.cc
diff --git a/chrome/browser/chromeos/login/login_utils.cc b/chrome/browser/chromeos/login/login_utils.cc
index 80c76fa43b4ffec72bc79d40a387acc68f44f043..aad2153052a5d280aea5ae25bae4c60c848e4d06 100644
--- a/chrome/browser/chromeos/login/login_utils.cc
+++ b/chrome/browser/chromeos/login/login_utils.cc
@@ -215,9 +215,7 @@ class LoginUtilsImpl : public LoginUtils,
static void RunCallbackOnLocaleLoaded(
const base::Closure& callback,
InputEventsBlocker* input_events_blocker,
- const std::string& locale,
- const std::string& loaded_locale,
- const bool success);
+ scoped_ptr<locale_util::LanguageSwitchResult> result);
// Attempts restarting the browser process and esures that this does
// not happen while we are still fetching new OAuth refresh tokens.
@@ -328,9 +326,7 @@ void LoginUtilsImpl::DoBrowserLaunchInternal(Profile* profile,
void LoginUtilsImpl::RunCallbackOnLocaleLoaded(
const base::Closure& callback,
InputEventsBlocker* /* input_events_blocker */,
- const std::string& /* locale */,
- const std::string& /* loaded_locale */,
- const bool /* success */) {
+ scoped_ptr<locale_util::LanguageSwitchResult> /* result */) {
callback.Run();
}

Powered by Google App Engine
This is Rietveld 408576698