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

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

Issue 2642823011: cros: Fold NetworkErrorModel into NetworkErrorView. (Closed)
Patch Set: Created 3 years, 11 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 b3092a57d6473f69e9170a80da3d7c198c79f016..21b0ee4bd9477a779c9373eacfbaceea6d591abe 100644
--- a/chrome/browser/ui/webui/chromeos/login/error_screen_handler.cc
+++ b/chrome/browser/ui/webui/chromeos/login/error_screen_handler.cc
@@ -7,7 +7,7 @@
#include "ash/common/system/chromeos/devicetype_utils.h"
#include "base/message_loop/message_loop.h"
#include "base/time/time.h"
-#include "chrome/browser/chromeos/login/screens/network_error_model.h"
+#include "chrome/browser/chromeos/login/screens/error_screen.h"
#include "chrome/grit/chromium_strings.h"
#include "chrome/grit/generated_resources.h"
#include "components/login/localized_values_builder.h"
@@ -23,15 +23,12 @@ namespace chromeos {
ErrorScreenHandler::ErrorScreenHandler()
: BaseScreenHandler(kJsScreenPath),
- model_(nullptr),
- show_on_init_(false),
- showing_(false),
weak_ptr_factory_(this) {
}
ErrorScreenHandler::~ErrorScreenHandler() {
- if (model_)
- model_->OnViewDestroyed(this);
+ if (screen_)
+ screen_->OnViewDestroyed(this);
}
void ErrorScreenHandler::Show() {
@@ -40,24 +37,24 @@ void ErrorScreenHandler::Show() {
return;
}
BaseScreenHandler::ShowScreen(OobeScreen::SCREEN_ERROR_MESSAGE);
- if (model_)
- model_->OnShow();
+ if (screen_)
+ screen_->OnShow();
showing_ = true;
}
void ErrorScreenHandler::Hide() {
showing_ = false;
- if (model_)
- model_->OnHide();
+ if (screen_)
+ screen_->OnHide();
}
-void ErrorScreenHandler::Bind(NetworkErrorModel& model) {
- model_ = &model;
- BaseScreenHandler::SetBaseScreen(model_);
+void ErrorScreenHandler::Bind(ErrorScreen* screen) {
+ screen_ = screen;
+ BaseScreenHandler::SetBaseScreen(screen_);
}
void ErrorScreenHandler::Unbind() {
- model_ = nullptr;
+ screen_ = nullptr;
BaseScreenHandler::SetBaseScreen(nullptr);
}
@@ -66,8 +63,8 @@ void ErrorScreenHandler::ShowOobeScreen(OobeScreen screen) {
}
void ErrorScreenHandler::HandleHideCaptivePortal() {
- if (model_)
- model_->HideCaptivePortal();
+ if (screen_)
+ screen_->HideCaptivePortal();
}
void ErrorScreenHandler::RegisterMessages() {
@@ -127,8 +124,8 @@ void ErrorScreenHandler::Initialize() {
}
void ErrorScreenHandler::OnConnectToNetworkRequested() {
- if (showing_ && model_)
- model_->OnUserAction(NetworkErrorModel::kUserActionConnectRequested);
+ if (showing_ && screen_)
+ screen_->OnUserAction(ErrorScreen::kUserActionConnectRequested);
}
} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698