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

Side by Side Diff: components/ntp_snippets/ntp_snippets_status_service.cc

Issue 2158373002: [NTP Snippets] Add a pref to disable the NTPSnippetsService (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 5 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 "components/ntp_snippets/ntp_snippets_status_service.h" 5 #include "components/ntp_snippets/ntp_snippets_status_service.h"
6
7 #include "components/ntp_snippets/pref_names.h"
8 #include "components/prefs/pref_registry_simple.h"
9 #include "components/prefs/pref_service.h"
6 #include "components/signin/core/browser/signin_manager.h" 10 #include "components/signin/core/browser/signin_manager.h"
7 #include "components/sync_driver/sync_service.h" 11 #include "components/sync_driver/sync_service.h"
8 12
9 namespace ntp_snippets { 13 namespace ntp_snippets {
10 14
11 NTPSnippetsStatusService::NTPSnippetsStatusService( 15 NTPSnippetsStatusService::NTPSnippetsStatusService(
12 SigninManagerBase* signin_manager, 16 SigninManagerBase* signin_manager,
13 sync_driver::SyncService* sync_service) 17 sync_driver::SyncService* sync_service,
18 PrefService* pref_service)
14 : disabled_reason_(DisabledReason::EXPLICITLY_DISABLED), 19 : disabled_reason_(DisabledReason::EXPLICITLY_DISABLED),
15 signin_manager_(signin_manager), 20 signin_manager_(signin_manager),
16 sync_service_(sync_service), 21 sync_service_(sync_service),
22 pref_service_(pref_service),
17 sync_service_observer_(this) {} 23 sync_service_observer_(this) {}
18 24
19 NTPSnippetsStatusService::~NTPSnippetsStatusService() {} 25 NTPSnippetsStatusService::~NTPSnippetsStatusService() {}
20 26
27 // static
28 void NTPSnippetsStatusService::RegisterProfilePrefs(
29 PrefRegistrySimple* registry) {
30 registry->RegisterBooleanPref(prefs::kDisableSnippets, false);
31 }
32
21 void NTPSnippetsStatusService::Init( 33 void NTPSnippetsStatusService::Init(
22 const DisabledReasonChangeCallback& callback) { 34 const DisabledReasonChangeCallback& callback) {
23 DCHECK(disabled_reason_change_callback_.is_null()); 35 DCHECK(disabled_reason_change_callback_.is_null());
24 36
25 disabled_reason_change_callback_ = callback; 37 disabled_reason_change_callback_ = callback;
26 38
27 // Notify about the current state before registering the observer, to make 39 // Notify about the current state before registering the observer, to make
28 // sure we don't get a double notification due to an undefined start state. 40 // sure we don't get a double notification due to an undefined start state.
29 disabled_reason_ = GetDisabledReasonFromDeps(); 41 disabled_reason_ = GetDisabledReasonFromDeps();
30 disabled_reason_change_callback_.Run(disabled_reason_); 42 disabled_reason_change_callback_.Run(disabled_reason_);
31 43
32 sync_service_observer_.Add(sync_service_); 44 sync_service_observer_.Add(sync_service_);
45
46 pref_change_registrar_.Init(pref_service_);
47 pref_change_registrar_.Add(
48 prefs::kDisableSnippets,
49 base::Bind(&NTPSnippetsStatusService::OnStateChanged,
50 base::Unretained(this)));
33 } 51 }
34 52
35 void NTPSnippetsStatusService::OnStateChanged() { 53 void NTPSnippetsStatusService::OnStateChanged() {
36 DisabledReason new_disabled_reason = GetDisabledReasonFromDeps(); 54 DisabledReason new_disabled_reason = GetDisabledReasonFromDeps();
37 55
38 if (new_disabled_reason == disabled_reason_) 56 if (new_disabled_reason == disabled_reason_)
39 return; 57 return;
40 58
41 disabled_reason_ = new_disabled_reason; 59 disabled_reason_ = new_disabled_reason;
42 disabled_reason_change_callback_.Run(disabled_reason_); 60 disabled_reason_change_callback_.Run(disabled_reason_);
43 } 61 }
44 62
45 DisabledReason NTPSnippetsStatusService::GetDisabledReasonFromDeps() const { 63 DisabledReason NTPSnippetsStatusService::GetDisabledReasonFromDeps() const {
64 if (pref_service_->GetBoolean(prefs::kDisableSnippets)) {
65 DVLOG(1) << "[GetNewDisabledReason] Disabled via pref";
66 return DisabledReason::EXPLICITLY_DISABLED;
67 }
68
46 if (!signin_manager_ || !signin_manager_->IsAuthenticated()) { 69 if (!signin_manager_ || !signin_manager_->IsAuthenticated()) {
47 DVLOG(1) << "[GetNewDisabledReason] Signed out"; 70 DVLOG(1) << "[GetNewDisabledReason] Signed out";
48 return DisabledReason::SIGNED_OUT; 71 return DisabledReason::SIGNED_OUT;
49 } 72 }
50 73
51 if (!sync_service_ || !sync_service_->CanSyncStart()) { 74 if (!sync_service_ || !sync_service_->CanSyncStart()) {
52 DVLOG(1) << "[GetNewDisabledReason] Sync disabled"; 75 DVLOG(1) << "[GetNewDisabledReason] Sync disabled";
53 return DisabledReason::SYNC_DISABLED; 76 return DisabledReason::SYNC_DISABLED;
54 } 77 }
55 78
(...skipping 15 matching lines...) Expand all
71 syncer::HISTORY_DELETE_DIRECTIVES)) { 94 syncer::HISTORY_DELETE_DIRECTIVES)) {
72 DVLOG(1) << "[GetNewDisabledReason] History sync disabled"; 95 DVLOG(1) << "[GetNewDisabledReason] History sync disabled";
73 return DisabledReason::HISTORY_SYNC_DISABLED; 96 return DisabledReason::HISTORY_SYNC_DISABLED;
74 } 97 }
75 98
76 DVLOG(1) << "[GetNewDisabledReason] Enabled"; 99 DVLOG(1) << "[GetNewDisabledReason] Enabled";
77 return DisabledReason::NONE; 100 return DisabledReason::NONE;
78 } 101 }
79 102
80 } // namespace ntp_snippets 103 } // namespace ntp_snippets
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698