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

Side by Side Diff: chrome/browser/chromeos/login/enrollment/auto_enrollment_check_screen.cc

Issue 539483002: Move all typecasting code to individual screens. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/enrollment/auto_enrollment_check_screen. h" 5 #include "chrome/browser/chromeos/login/enrollment/auto_enrollment_check_screen. h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/bind_helpers.h" 8 #include "base/bind_helpers.h"
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
11 #include "chrome/browser/chromeos/login/screens/screen_observer.h" 11 #include "chrome/browser/chromeos/login/screens/screen_observer.h"
12 #include "chrome/browser/chromeos/login/wizard_controller.h" 12 #include "chrome/browser/chromeos/login/wizard_controller.h"
13 #include "chromeos/chromeos_switches.h" 13 #include "chromeos/chromeos_switches.h"
14 #include "chromeos/network/network_state.h" 14 #include "chromeos/network/network_state.h"
15 #include "chromeos/network/network_state_handler.h" 15 #include "chromeos/network/network_state_handler.h"
16 16
17 namespace chromeos { 17 namespace chromeos {
18 18
19 // static
ygorshenin1 2014/09/03 12:58:46 Static methods definitions usually go after ctor a
Denis Kuznetsov (DE-MUC) 2014/09/03 13:16:05 Acknowledged.
20 AutoEnrollmentCheckScreen* AutoEnrollmentCheckScreen::Get(
21 ScreenManager* manager) {
ygorshenin1 2014/09/03 12:58:46 Please, add inclusion of chrome/browser/chromeos/l
Denis Kuznetsov (DE-MUC) 2014/09/03 13:16:05 Done.
22 return static_cast<AutoEnrollmentCheckScreen*>(
23 manager->GetScreen(WizardController::kAutoEnrollmentCheckScreenName));
24 }
25
19 AutoEnrollmentCheckScreen::AutoEnrollmentCheckScreen( 26 AutoEnrollmentCheckScreen::AutoEnrollmentCheckScreen(
20 ScreenObserver* observer, 27 ScreenObserver* observer,
21 AutoEnrollmentCheckScreenActor* actor) 28 AutoEnrollmentCheckScreenActor* actor)
22 : WizardScreen(observer), 29 : WizardScreen(observer),
23 actor_(actor), 30 actor_(actor),
24 captive_portal_status_( 31 captive_portal_status_(
25 NetworkPortalDetector::CAPTIVE_PORTAL_STATUS_UNKNOWN), 32 NetworkPortalDetector::CAPTIVE_PORTAL_STATUS_UNKNOWN),
26 auto_enrollment_state_(policy::AUTO_ENROLLMENT_STATE_IDLE) { 33 auto_enrollment_state_(policy::AUTO_ENROLLMENT_STATE_IDLE) {
27 if (actor_) 34 if (actor_)
28 actor_->SetDelegate(this); 35 actor_->SetDelegate(this);
(...skipping 185 matching lines...) Expand 10 before | Expand all | Expand 10 after
214 } 221 }
215 222
216 void AutoEnrollmentCheckScreen::SignalCompletion() { 223 void AutoEnrollmentCheckScreen::SignalCompletion() {
217 NetworkPortalDetector::Get()->RemoveObserver(this); 224 NetworkPortalDetector::Get()->RemoveObserver(this);
218 auto_enrollment_progress_subscription_.reset(); 225 auto_enrollment_progress_subscription_.reset();
219 get_screen_observer()->OnExit( 226 get_screen_observer()->OnExit(
220 ScreenObserver::ENTERPRISE_AUTO_ENROLLMENT_CHECK_COMPLETED); 227 ScreenObserver::ENTERPRISE_AUTO_ENROLLMENT_CHECK_COMPLETED);
221 } 228 }
222 229
223 } // namespace chromeos 230 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698