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

Unified Diff: chrome/browser/ui/webui/chromeos/login/oobe_ui.cc

Issue 1098303002: Fix crash on Enrollment screen. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix test compilation. 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/webui/chromeos/login/oobe_ui.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/webui/chromeos/login/oobe_ui.cc
diff --git a/chrome/browser/ui/webui/chromeos/login/oobe_ui.cc b/chrome/browser/ui/webui/chromeos/login/oobe_ui.cc
index 2f3a1cb03b3c33b34e9a200387c225b0fa257343..b271920a68e70fc73cb5b5886a269acafdd493fa 100644
--- a/chrome/browser/ui/webui/chromeos/login/oobe_ui.cc
+++ b/chrome/browser/ui/webui/chromeos/login/oobe_ui.cc
@@ -294,18 +294,8 @@ OobeUI::OobeUI(content::WebUI* web_ui, const GURL& url)
error_screen_handler_ = new ErrorScreenHandler();
AddScreenHandler(error_screen_handler_);
- // Initialize ErrorScreen if it hasn't initialized so that NetworkErrorModel
- // is binded properly.
- NetworkErrorModel* network_error_model = nullptr;
- if (WizardController::default_controller()) {
- network_error_model = static_cast<NetworkErrorModel*>(
- WizardController::default_controller()->GetScreen(
- WizardController::kErrorScreenName));
- CHECK(network_error_model);
- } else {
- error_screen_.reset(new ErrorScreen(nullptr, error_screen_handler_));
- network_error_model = error_screen_.get();
- }
+ error_screen_.reset(new ErrorScreen(nullptr, error_screen_handler_));
+ NetworkErrorModel* network_error_model = error_screen_.get();
EnrollmentScreenHandler* enrollment_screen_handler =
new EnrollmentScreenHandler(network_state_informer_, network_error_model);
@@ -468,8 +458,8 @@ UserImageView* OobeUI::GetUserImageView() {
return user_image_view_;
}
-NetworkErrorView* OobeUI::GetNetworkErrorView() {
- return error_screen_handler_;
+ErrorScreen* OobeUI::GetErrorScreen() {
+ return error_screen_.get();
}
SupervisedUserCreationScreenHandler*
@@ -527,10 +517,6 @@ void OobeUI::GetLocalizedStrings(base::DictionaryValue* localized_strings) {
localized_strings->SetString("newKioskUI", new_kiosk_ui ? "on" : "off");
}
-scoped_ptr<ErrorScreen> OobeUI::GetErrorScreen() {
- return error_screen_.Pass();
-}
-
void OobeUI::InitializeScreenMaps() {
screen_names_.resize(SCREEN_UNKNOWN);
screen_names_[SCREEN_OOBE_HID_DETECTION] = kScreenOobeHIDDetection;
« no previous file with comments | « chrome/browser/ui/webui/chromeos/login/oobe_ui.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698