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

Side by Side Diff: chrome/browser/chromeos/login/wizard_controller.cc

Issue 276433003: UMA metrics added for HID detection dialog. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Switched to bool histogram. Added Continue-scenario and HIDShows per OOBE completion histograms. Created 6 years, 7 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/wizard_controller.h" 5 #include "chrome/browser/chromeos/login/wizard_controller.h"
6 6
7 #include <signal.h> 7 #include <signal.h>
8 #include <stdlib.h> 8 #include <stdlib.h>
9 #include <sys/types.h> 9 #include <sys/types.h>
10 10
(...skipping 692 matching lines...) Expand 10 before | Expand all | Expand 10 after
703 // ChromiumOS builds would go though this code path too. 703 // ChromiumOS builds would go though this code path too.
704 NetworkHandler::Get()->network_state_handler()->SetCheckPortalList( 704 NetworkHandler::Get()->network_state_handler()->SetCheckPortalList(
705 NetworkStateHandler::kDefaultCheckPortalList); 705 NetworkStateHandler::kDefaultCheckPortalList);
706 host_->GetAutoEnrollmentController()->Start(); 706 host_->GetAutoEnrollmentController()->Start();
707 host_->PrewarmAuthentication(); 707 host_->PrewarmAuthentication();
708 NetworkPortalDetector::Get()->Enable(true); 708 NetworkPortalDetector::Get()->Enable(true);
709 } 709 }
710 710
711 void WizardController::PerformOOBECompletedActions() { 711 void WizardController::PerformOOBECompletedActions() {
712 StartupUtils::MarkOobeCompleted(); 712 StartupUtils::MarkOobeCompleted();
713 UMA_HISTOGRAM_COUNTS_100(
714 "HIDDetection.TimesDialogShownPerOOBECompleted",
715 GetLocalState()->GetInteger(prefs::kTimesHIDDialogShown));
Mattias Nissler (ping if slow) 2014/05/12 11:10:02 Would it make sense to clear the pref here? I susp
merkulova 2014/05/13 07:18:01 Done.
713 } 716 }
714 717
715 void WizardController::SetCurrentScreen(WizardScreen* new_current) { 718 void WizardController::SetCurrentScreen(WizardScreen* new_current) {
716 SetCurrentScreenSmooth(new_current, false); 719 SetCurrentScreenSmooth(new_current, false);
717 } 720 }
718 721
719 void WizardController::ShowCurrentScreen() { 722 void WizardController::ShowCurrentScreen() {
720 // ShowCurrentScreen may get called by smooth_show_timer_ even after 723 // ShowCurrentScreen may get called by smooth_show_timer_ even after
721 // flow has been switched to sign in screen (ExistingUserController). 724 // flow has been switched to sign in screen (ExistingUserController).
722 if (!oobe_display_) 725 if (!oobe_display_)
(...skipping 337 matching lines...) Expand 10 before | Expand all | Expand 10 after
1060 // cancelled on destruction. 1063 // cancelled on destruction.
1061 GetTimezoneProvider()->RequestTimezone( 1064 GetTimezoneProvider()->RequestTimezone(
1062 position, 1065 position,
1063 false, // sensor 1066 false, // sensor
1064 timeout - elapsed, 1067 timeout - elapsed,
1065 base::Bind(&WizardController::OnTimezoneResolved, 1068 base::Bind(&WizardController::OnTimezoneResolved,
1066 base::Unretained(this))); 1069 base::Unretained(this)));
1067 } 1070 }
1068 1071
1069 } // namespace chromeos 1072 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698