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

Unified Diff: chrome/browser/chromeos/login/enrollment/auto_enrollment_check_screen.cc

Issue 679133002: ScreenObserver is renamed to BaseScreenDelegate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixes. 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/enrollment/auto_enrollment_check_screen.cc
diff --git a/chrome/browser/chromeos/login/enrollment/auto_enrollment_check_screen.cc b/chrome/browser/chromeos/login/enrollment/auto_enrollment_check_screen.cc
index 87f10acfa36e2c778bc2e9da115852c5c99572a4..a7bff56e7bc1f55c885d1ab9369128c4a63d1e96 100644
--- a/chrome/browser/chromeos/login/enrollment/auto_enrollment_check_screen.cc
+++ b/chrome/browser/chromeos/login/enrollment/auto_enrollment_check_screen.cc
@@ -10,7 +10,7 @@
#include "base/logging.h"
#include "chrome/browser/chromeos/login/error_screens_histogram_helper.h"
#include "chrome/browser/chromeos/login/screen_manager.h"
-#include "chrome/browser/chromeos/login/screens/screen_observer.h"
+#include "chrome/browser/chromeos/login/screens/base_screen_delegate.h"
#include "chrome/browser/chromeos/login/wizard_controller.h"
#include "chromeos/chromeos_switches.h"
#include "chromeos/network/network_state.h"
@@ -26,9 +26,9 @@ AutoEnrollmentCheckScreen* AutoEnrollmentCheckScreen::Get(
}
AutoEnrollmentCheckScreen::AutoEnrollmentCheckScreen(
- ScreenObserver* observer,
+ BaseScreenDelegate* base_screen_delegate,
AutoEnrollmentCheckScreenActor* actor)
- : BaseScreen(observer),
+ : BaseScreen(base_screen_delegate),
actor_(actor),
captive_portal_status_(
NetworkPortalDetector::CAPTIVE_PORTAL_STATUS_UNKNOWN),
@@ -99,8 +99,8 @@ std::string AutoEnrollmentCheckScreen::GetName() const {
}
void AutoEnrollmentCheckScreen::OnExit() {
- get_screen_observer()->OnExit(
- ScreenObserver::ENTERPRISE_AUTO_ENROLLMENT_CHECK_COMPLETED);
+ get_base_screen_delegate()->OnExit(
+ BaseScreenDelegate::ENTERPRISE_AUTO_ENROLLMENT_CHECK_COMPLETED);
}
void AutoEnrollmentCheckScreen::OnActorDestroyed(
@@ -128,7 +128,7 @@ void AutoEnrollmentCheckScreen::UpdateState(
UpdateAutoEnrollmentState(new_auto_enrollment_state);
// Update the connecting indicator.
- ErrorScreen* error_screen = get_screen_observer()->GetErrorScreen();
+ ErrorScreen* error_screen = get_base_screen_delegate()->GetErrorScreen();
error_screen->ShowConnectingIndicator(
new_auto_enrollment_state == policy::AUTO_ENROLLMENT_STATE_PENDING);
@@ -177,7 +177,7 @@ bool AutoEnrollmentCheckScreen::UpdateCaptivePortalStatus(
case NetworkPortalDetector::CAPTIVE_PORTAL_STATUS_PORTAL:
ShowErrorScreen(ErrorScreen::ERROR_STATE_PORTAL);
if (captive_portal_status_ != new_captive_portal_status)
- get_screen_observer()->GetErrorScreen()->FixCaptivePortal();
+ get_base_screen_delegate()->GetErrorScreen()->FixCaptivePortal();
return true;
case NetworkPortalDetector::CAPTIVE_PORTAL_STATUS_PROXY_AUTH_REQUIRED:
ShowErrorScreen(ErrorScreen::ERROR_STATE_PROXY);
@@ -218,20 +218,20 @@ void AutoEnrollmentCheckScreen::ShowErrorScreen(
ErrorScreen::ErrorState error_state) {
const NetworkState* network =
NetworkHandler::Get()->network_state_handler()->DefaultNetwork();
- ErrorScreen* error_screen = get_screen_observer()->GetErrorScreen();
+ ErrorScreen* error_screen = get_base_screen_delegate()->GetErrorScreen();
error_screen->SetUIState(ErrorScreen::UI_STATE_AUTO_ENROLLMENT_ERROR);
error_screen->AllowGuestSignin(true);
error_screen->SetErrorState(error_state,
network ? network->name() : std::string());
- get_screen_observer()->ShowErrorScreen();
+ get_base_screen_delegate()->ShowErrorScreen();
histogram_helper_->OnErrorShow(error_state);
}
void AutoEnrollmentCheckScreen::SignalCompletion() {
NetworkPortalDetector::Get()->RemoveObserver(this);
auto_enrollment_progress_subscription_.reset();
- get_screen_observer()->OnExit(
- ScreenObserver::ENTERPRISE_AUTO_ENROLLMENT_CHECK_COMPLETED);
+ get_base_screen_delegate()->OnExit(
+ BaseScreenDelegate::ENTERPRISE_AUTO_ENROLLMENT_CHECK_COMPLETED);
}
} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698