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

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

Issue 2713513009: cros: Break BaseScreenHandler into two classes. (Closed)
Patch Set: Add comments, fix compile Created 3 years, 10 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/ui/webui/chromeos/login/error_screen_handler.cc
diff --git a/chrome/browser/ui/webui/chromeos/login/error_screen_handler.cc b/chrome/browser/ui/webui/chromeos/login/error_screen_handler.cc
index 5f0d5895df6cc4bfc48ce409133ed212f5d688e6..e93b4ddc1268ec9200e434702193de43e56fe345 100644
--- a/chrome/browser/ui/webui/chromeos/login/error_screen_handler.cc
+++ b/chrome/browser/ui/webui/chromeos/login/error_screen_handler.cc
@@ -21,7 +21,8 @@ const char kJsScreenPath[] = "login.ErrorMessageScreen";
namespace chromeos {
-ErrorScreenHandler::ErrorScreenHandler() : weak_ptr_factory_(this) {
+ErrorScreenHandler::ErrorScreenHandler()
+ : BaseScreenHandler(kScreenId), weak_ptr_factory_(this) {
set_call_js_prefix(kJsScreenPath);
}
@@ -35,7 +36,7 @@ void ErrorScreenHandler::Show() {
show_on_init_ = true;
return;
}
- BaseScreenHandler::ShowScreen(OobeScreen::SCREEN_ERROR_MESSAGE);
+ BaseScreenHandler::ShowScreen(kScreenId);
if (screen_)
screen_->OnShow();
showing_ = true;

Powered by Google App Engine
This is Rietveld 408576698