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

Side by Side Diff: chrome/browser/chromeos/login/screens/network_screen.cc

Issue 1049873005: [chrome/browser/chromeos/] favor DCHECK_CURRENTLY_ON for better logs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/chromeos/login/screens/network_screen.h" 5 #include "chrome/browser/chromeos/login/screens/network_screen.h"
6 6
7 #include "base/location.h" 7 #include "base/location.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/prefs/pref_service.h" 9 #include "base/prefs/pref_service.h"
10 #include "base/strings/string16.h" 10 #include "base/strings/string16.h"
(...skipping 352 matching lines...) Expand 10 before | Expand all | Expand 10 after
363 scoped_ptr<locale_util::LanguageSwitchResult> language_switch_result) { 363 scoped_ptr<locale_util::LanguageSwitchResult> language_switch_result) {
364 UILanguageListResolvedCallback callback = base::Bind( 364 UILanguageListResolvedCallback callback = base::Bind(
365 &NetworkScreen::OnLanguageListResolved, weak_factory_.GetWeakPtr()); 365 &NetworkScreen::OnLanguageListResolved, weak_factory_.GetWeakPtr());
366 ResolveUILanguageList(language_switch_result.Pass(), callback); 366 ResolveUILanguageList(language_switch_result.Pass(), callback);
367 } 367 }
368 368
369 void NetworkScreen::OnLanguageListResolved( 369 void NetworkScreen::OnLanguageListResolved(
370 scoped_ptr<base::ListValue> new_language_list, 370 scoped_ptr<base::ListValue> new_language_list,
371 std::string new_language_list_locale, 371 std::string new_language_list_locale,
372 std::string new_selected_language) { 372 std::string new_selected_language) {
373 DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI)); 373 DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
374 374
375 language_list_.reset(new_language_list.release()); 375 language_list_.reset(new_language_list.release());
376 language_list_locale_ = new_language_list_locale; 376 language_list_locale_ = new_language_list_locale;
377 selected_language_code_ = new_selected_language; 377 selected_language_code_ = new_selected_language;
378 378
379 g_browser_process->local_state()->SetString(prefs::kApplicationLocale, 379 g_browser_process->local_state()->SetString(prefs::kApplicationLocale,
380 selected_language_code_); 380 selected_language_code_);
381 if (view_) 381 if (view_)
382 view_->ReloadLocalizedContent(); 382 view_->ReloadLocalizedContent();
383 FOR_EACH_OBSERVER(Observer, observers_, OnLanguageListReloaded()); 383 FOR_EACH_OBSERVER(Observer, observers_, OnLanguageListReloaded());
384 } 384 }
385 385
386 void NetworkScreen::OnSystemTimezoneChanged() { 386 void NetworkScreen::OnSystemTimezoneChanged() {
387 std::string current_timezone_id; 387 std::string current_timezone_id;
388 CrosSettings::Get()->GetString(kSystemTimezone, &current_timezone_id); 388 CrosSettings::Get()->GetString(kSystemTimezone, &current_timezone_id);
389 GetContextEditor().SetString(kContextKeyTimezone, current_timezone_id); 389 GetContextEditor().SetString(kContextKeyTimezone, current_timezone_id);
390 } 390 }
391 391
392 } // namespace chromeos 392 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698