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

Side by Side Diff: chrome/browser/chromeos/login/screens/arc_terms_of_service_screen.cc

Issue 2549373005: cros: Refactor of BaseScreen::GetName to BaseScreen::screen_id. (Closed)
Patch Set: Nits 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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/screens/arc_terms_of_service_screen.h" 5 #include "chrome/browser/chromeos/login/screens/arc_terms_of_service_screen.h"
6 6
7 #include "chrome/browser/chromeos/login/screens/base_screen_delegate.h" 7 #include "chrome/browser/chromeos/login/screens/base_screen_delegate.h"
8 #include "chrome/browser/chromeos/login/wizard_controller.h" 8 #include "chrome/browser/chromeos/login/wizard_controller.h"
9 #include "chrome/browser/metrics/metrics_reporting_state.h" 9 #include "chrome/browser/metrics/metrics_reporting_state.h"
10 #include "chrome/browser/profiles/profile.h" 10 #include "chrome/browser/profiles/profile.h"
11 #include "chrome/browser/profiles/profile_manager.h" 11 #include "chrome/browser/profiles/profile_manager.h"
12 #include "chrome/common/pref_names.h" 12 #include "chrome/common/pref_names.h"
13 #include "components/prefs/pref_service.h" 13 #include "components/prefs/pref_service.h"
14 14
15 namespace chromeos { 15 namespace chromeos {
16 16
17 ArcTermsOfServiceScreen::ArcTermsOfServiceScreen( 17 ArcTermsOfServiceScreen::ArcTermsOfServiceScreen(
18 BaseScreenDelegate* base_screen_delegate, 18 BaseScreenDelegate* base_screen_delegate,
19 ArcTermsOfServiceScreenActor* actor) 19 ArcTermsOfServiceScreenActor* actor)
20 : BaseScreen(base_screen_delegate), actor_(actor) { 20 : BaseScreen(base_screen_delegate,
21 WizardController::kArcTermsOfServiceScreenName),
22 actor_(actor) {
21 DCHECK(actor_); 23 DCHECK(actor_);
22 if (actor_) 24 if (actor_)
23 actor_->SetDelegate(this); 25 actor_->SetDelegate(this);
24 } 26 }
25 27
26 ArcTermsOfServiceScreen::~ArcTermsOfServiceScreen() { 28 ArcTermsOfServiceScreen::~ArcTermsOfServiceScreen() {
27 if (actor_) 29 if (actor_)
28 actor_->SetDelegate(nullptr); 30 actor_->SetDelegate(nullptr);
29 } 31 }
30 32
31 void ArcTermsOfServiceScreen::Show() { 33 void ArcTermsOfServiceScreen::Show() {
32 if (!actor_) 34 if (!actor_)
33 return; 35 return;
34 36
35 // Show the screen. 37 // Show the screen.
36 actor_->Show(); 38 actor_->Show();
37 } 39 }
38 40
39 void ArcTermsOfServiceScreen::Hide() { 41 void ArcTermsOfServiceScreen::Hide() {
40 if (actor_) 42 if (actor_)
41 actor_->Hide(); 43 actor_->Hide();
42 } 44 }
43 45
44 std::string ArcTermsOfServiceScreen::GetName() const {
45 return WizardController::kArcTermsOfServiceScreenName;
46 }
47
48 void ArcTermsOfServiceScreen::OnSkip() { 46 void ArcTermsOfServiceScreen::OnSkip() {
49 ApplyTerms(false); 47 ApplyTerms(false);
50 } 48 }
51 49
52 void ArcTermsOfServiceScreen::OnAccept() { 50 void ArcTermsOfServiceScreen::OnAccept() {
53 ApplyTerms(true); 51 ApplyTerms(true);
54 } 52 }
55 53
56 void ArcTermsOfServiceScreen::ApplyTerms(bool accepted) { 54 void ArcTermsOfServiceScreen::ApplyTerms(bool accepted) {
57 Profile* profile = ProfileManager::GetActiveUserProfile(); 55 Profile* profile = ProfileManager::GetActiveUserProfile();
58 profile->GetPrefs()->SetBoolean(prefs::kArcTermsAccepted, accepted); 56 profile->GetPrefs()->SetBoolean(prefs::kArcTermsAccepted, accepted);
59 profile->GetPrefs()->SetBoolean(prefs::kArcEnabled, accepted); 57 profile->GetPrefs()->SetBoolean(prefs::kArcEnabled, accepted);
60 58
61 Finish(BaseScreenDelegate::ARC_TERMS_OF_SERVICE_FINISHED); 59 Finish(BaseScreenDelegate::ARC_TERMS_OF_SERVICE_FINISHED);
62 } 60 }
63 61
64 void ArcTermsOfServiceScreen::OnActorDestroyed( 62 void ArcTermsOfServiceScreen::OnActorDestroyed(
65 ArcTermsOfServiceScreenActor* actor) { 63 ArcTermsOfServiceScreenActor* actor) {
66 DCHECK_EQ(actor, actor_); 64 DCHECK_EQ(actor, actor_);
67 actor_ = nullptr; 65 actor_ = nullptr;
68 } 66 }
69 67
70 } // namespace chromeos 68 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698