[NTP Snippets] Replace the "DisableSnippets" pref by an "Enable" one

components/ntp_snippets/ntp_snippets_status_service.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/ntp_snippets/ntp_snippets_status_service.h"
 
 #include "components/ntp_snippets/pref_names.h"
 #include "components/prefs/pref_registry_simple.h"
 #include "components/prefs/pref_service.h"
 #include "components/signin/core/browser/signin_manager.h"
 #include "components/sync_driver/sync_service.h"
 
 namespace ntp_snippets {
 
 NTPSnippetsStatusService::NTPSnippetsStatusService(
     SigninManagerBase* signin_manager,
     sync_driver::SyncService* sync_service,
     PrefService* pref_service)
     : disabled_reason_(DisabledReason::EXPLICITLY_DISABLED),
       signin_manager_(signin_manager),
       sync_service_(sync_service),
       pref_service_(pref_service),
       sync_service_observer_(this) {}
 
 NTPSnippetsStatusService::~NTPSnippetsStatusService() {}
 
 // static
 void NTPSnippetsStatusService::RegisterProfilePrefs(
     PrefRegistrySimple* registry) {
-  registry->RegisterBooleanPref(prefs::kDisableSnippets, false);
+  registry->RegisterBooleanPref(prefs::kEnableSnippets, true);
 }
 
 void NTPSnippetsStatusService::Init(
     const DisabledReasonChangeCallback& callback) {
   DCHECK(disabled_reason_change_callback_.is_null());
 
   disabled_reason_change_callback_ = callback;
 
   // Notify about the current state before registering the observer, to make
   // sure we don't get a double notification due to an undefined start state.
   disabled_reason_ = GetDisabledReasonFromDeps();
   disabled_reason_change_callback_.Run(disabled_reason_);
 
   sync_service_observer_.Add(sync_service_);
 
   pref_change_registrar_.Init(pref_service_);
   pref_change_registrar_.Add(
-      prefs::kDisableSnippets,
+      prefs::kEnableSnippets,
       base::Bind(&NTPSnippetsStatusService::OnStateChanged,
                  base::Unretained(this)));
 }
 
 void NTPSnippetsStatusService::OnStateChanged() {
   DisabledReason new_disabled_reason = GetDisabledReasonFromDeps();
 
   if (new_disabled_reason == disabled_reason_)
     return;
 
   disabled_reason_ = new_disabled_reason;
   disabled_reason_change_callback_.Run(disabled_reason_);
 }
 
 DisabledReason NTPSnippetsStatusService::GetDisabledReasonFromDeps() const {
-  if (pref_service_->GetBoolean(prefs::kDisableSnippets)) {
+  if (!pref_service_->GetBoolean(prefs::kEnableSnippets)) {
     DVLOG(1) << "[GetNewDisabledReason] Disabled via pref";
     return DisabledReason::EXPLICITLY_DISABLED;
   }
 
   if (!signin_manager_ || !signin_manager_->IsAuthenticated()) {
     DVLOG(1) << "[GetNewDisabledReason] Signed out";
     return DisabledReason::SIGNED_OUT;
   }
 
   if (!sync_service_ || !sync_service_->CanSyncStart()) {
@@ skipping 19 matching lines @@
           syncer::HISTORY_DELETE_DIRECTIVES)) {
     DVLOG(1) << "[GetNewDisabledReason] History sync disabled";
     return DisabledReason::HISTORY_SYNC_DISABLED;
   }
 
   DVLOG(1) << "[GetNewDisabledReason] Enabled";
   return DisabledReason::NONE;
 }
 
 }  // namespace ntp_snippets