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

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

Issue 614223006: Revert "UMA: How often are different network error screens encountered during OOBE" (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2171
Patch Set: 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 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/error_screens_histogram_helper.h"
12 #include "chrome/browser/chromeos/login/screen_manager.h" 11 #include "chrome/browser/chromeos/login/screen_manager.h"
13 #include "chrome/browser/chromeos/login/screens/screen_observer.h" 12 #include "chrome/browser/chromeos/login/screens/screen_observer.h"
14 #include "chrome/browser/chromeos/login/wizard_controller.h" 13 #include "chrome/browser/chromeos/login/wizard_controller.h"
15 #include "chromeos/chromeos_switches.h" 14 #include "chromeos/chromeos_switches.h"
16 #include "chromeos/network/network_state.h" 15 #include "chromeos/network/network_state.h"
17 #include "chromeos/network/network_state_handler.h" 16 #include "chromeos/network/network_state_handler.h"
18 17
19 namespace chromeos { 18 namespace chromeos {
20 19
21 // static 20 // static
22 AutoEnrollmentCheckScreen* AutoEnrollmentCheckScreen::Get( 21 AutoEnrollmentCheckScreen* AutoEnrollmentCheckScreen::Get(
23 ScreenManager* manager) { 22 ScreenManager* manager) {
24 return static_cast<AutoEnrollmentCheckScreen*>( 23 return static_cast<AutoEnrollmentCheckScreen*>(
25 manager->GetScreen(WizardController::kAutoEnrollmentCheckScreenName)); 24 manager->GetScreen(WizardController::kAutoEnrollmentCheckScreenName));
26 } 25 }
27 26
28 AutoEnrollmentCheckScreen::AutoEnrollmentCheckScreen( 27 AutoEnrollmentCheckScreen::AutoEnrollmentCheckScreen(
29 ScreenObserver* observer, 28 ScreenObserver* observer,
30 AutoEnrollmentCheckScreenActor* actor) 29 AutoEnrollmentCheckScreenActor* actor)
31 : WizardScreen(observer), 30 : WizardScreen(observer),
32 actor_(actor), 31 actor_(actor),
33 captive_portal_status_( 32 captive_portal_status_(
34 NetworkPortalDetector::CAPTIVE_PORTAL_STATUS_UNKNOWN), 33 NetworkPortalDetector::CAPTIVE_PORTAL_STATUS_UNKNOWN),
35 auto_enrollment_state_(policy::AUTO_ENROLLMENT_STATE_IDLE), 34 auto_enrollment_state_(policy::AUTO_ENROLLMENT_STATE_IDLE) {
36 histogram_helper_(new ErrorScreensHistogramHelper("Enrollment")) {
37 if (actor_) 35 if (actor_)
38 actor_->SetDelegate(this); 36 actor_->SetDelegate(this);
39 } 37 }
40 38
41 AutoEnrollmentCheckScreen::~AutoEnrollmentCheckScreen() { 39 AutoEnrollmentCheckScreen::~AutoEnrollmentCheckScreen() {
42 NetworkPortalDetector::Get()->RemoveObserver(this); 40 NetworkPortalDetector::Get()->RemoveObserver(this);
43 if (actor_) 41 if (actor_)
44 actor_->SetDelegate(NULL); 42 actor_->SetDelegate(NULL);
45 } 43 }
46 44
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
80 } 78 }
81 79
82 void AutoEnrollmentCheckScreen::PrepareToShow() { 80 void AutoEnrollmentCheckScreen::PrepareToShow() {
83 } 81 }
84 82
85 void AutoEnrollmentCheckScreen::Show() { 83 void AutoEnrollmentCheckScreen::Show() {
86 if (IsStartNeeded()) { 84 if (IsStartNeeded()) {
87 Start(); 85 Start();
88 if (actor_) 86 if (actor_)
89 actor_->Show(); 87 actor_->Show();
90 histogram_helper_->OnScreenShow();
91 } 88 }
92 } 89 }
93 90
94 void AutoEnrollmentCheckScreen::Hide() { 91 void AutoEnrollmentCheckScreen::Hide() {
95 } 92 }
96 93
97 std::string AutoEnrollmentCheckScreen::GetName() const { 94 std::string AutoEnrollmentCheckScreen::GetName() const {
98 return WizardController::kAutoEnrollmentCheckScreenName; 95 return WizardController::kAutoEnrollmentCheckScreenName;
99 } 96 }
100 97
(...skipping 116 matching lines...) Expand 10 before | Expand all | Expand 10 after
217 void AutoEnrollmentCheckScreen::ShowErrorScreen( 214 void AutoEnrollmentCheckScreen::ShowErrorScreen(
218 ErrorScreen::ErrorState error_state) { 215 ErrorScreen::ErrorState error_state) {
219 const NetworkState* network = 216 const NetworkState* network =
220 NetworkHandler::Get()->network_state_handler()->DefaultNetwork(); 217 NetworkHandler::Get()->network_state_handler()->DefaultNetwork();
221 ErrorScreen* error_screen = get_screen_observer()->GetErrorScreen(); 218 ErrorScreen* error_screen = get_screen_observer()->GetErrorScreen();
222 error_screen->SetUIState(ErrorScreen::UI_STATE_AUTO_ENROLLMENT_ERROR); 219 error_screen->SetUIState(ErrorScreen::UI_STATE_AUTO_ENROLLMENT_ERROR);
223 error_screen->AllowGuestSignin(true); 220 error_screen->AllowGuestSignin(true);
224 error_screen->SetErrorState(error_state, 221 error_screen->SetErrorState(error_state,
225 network ? network->name() : std::string()); 222 network ? network->name() : std::string());
226 get_screen_observer()->ShowErrorScreen(); 223 get_screen_observer()->ShowErrorScreen();
227 histogram_helper_->OnErrorShow(error_state);
228 } 224 }
229 225
230 void AutoEnrollmentCheckScreen::SignalCompletion() { 226 void AutoEnrollmentCheckScreen::SignalCompletion() {
231 NetworkPortalDetector::Get()->RemoveObserver(this); 227 NetworkPortalDetector::Get()->RemoveObserver(this);
232 auto_enrollment_progress_subscription_.reset(); 228 auto_enrollment_progress_subscription_.reset();
233 get_screen_observer()->OnExit( 229 get_screen_observer()->OnExit(
234 ScreenObserver::ENTERPRISE_AUTO_ENROLLMENT_CHECK_COMPLETED); 230 ScreenObserver::ENTERPRISE_AUTO_ENROLLMENT_CHECK_COMPLETED);
235 } 231 }
236 232
237 } // namespace chromeos 233 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698