📰 Let the backend trigger sign in related refreshes

components/ntp_snippets/remote/ntp_snippets_status_service.h

 // 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.
 
 #ifndef COMPONENTS_NTP_SNIPPETS_REMOTE_NTP_SNIPPETS_STATUS_SERVICE_H_
 #define COMPONENTS_NTP_SNIPPETS_REMOTE_NTP_SNIPPETS_STATUS_SERVICE_H_
 
 #include "base/callback.h"
 #include "base/gtest_prod_util.h"
 #include "base/scoped_observer.h"
 #include "components/prefs/pref_change_registrar.h"
-#include "components/signin/core/browser/signin_manager.h"
 
 class PrefRegistrySimple;
 class PrefService;
+class SigninManagerBase;
 
 namespace ntp_snippets {
 
 enum class SnippetsStatus : int {
   // Snippets are enabled and the user is signed in.
   ENABLED_AND_SIGNED_IN,
   // Snippets are enabled and the user is signed out (sign in is not required).
   ENABLED_AND_SIGNED_OUT,
   // Snippets have been disabled as part of the service configuration.
   EXPLICITLY_DISABLED,
   // The user is not signed in, and the service requires it to be enabled.
   SIGNED_OUT_AND_DISABLED,
 };
 
 // Aggregates data from preferences and signin to notify the snippet service of
 // relevant changes in their states.
-class NTPSnippetsStatusService : public SigninManagerBase::Observer {
+class NTPSnippetsStatusService {
  public:
   using SnippetsStatusChangeCallback =
       base::Callback<void(SnippetsStatus /*old_status*/,
                           SnippetsStatus /*new_status*/)>;
 
   NTPSnippetsStatusService(SigninManagerBase* signin_manager,
                            PrefService* pref_service);
 
-  ~NTPSnippetsStatusService() override;
+  virtual ~NTPSnippetsStatusService();
 
   static void RegisterProfilePrefs(PrefRegistrySimple* registry);
 
   // Starts listening for changes from the dependencies. |callback| will be
   // called when a significant change in state is detected.
   void Init(const SnippetsStatusChangeCallback& callback);
 
+  // To be called when the signin state changed. Will compute the new
+  // state considering the initialisation configuration and the preferences,
+  // and notify via the registered callback if appropriate.
+  void OnSignInStateChanged();
+
  private:
   FRIEND_TEST_ALL_PREFIXES(NTPSnippetsStatusServiceTest, DisabledViaPref);
 
-  // SigninManagerBase::Observer implementation
-  void GoogleSigninSucceeded(const std::string& account_id,
-                             const std::string& username,
-                             const std::string& password) override;
-  void GoogleSignedOut(const std::string& account_id,
-                       const std::string& username) override;
-
   // Callback for the PrefChangeRegistrar.
   void OnSnippetsEnabledChanged();
 
   void OnStateChanged(SnippetsStatus new_snippets_status);
 
   bool IsSignedIn() const;
 
   SnippetsStatus GetSnippetsStatusFromDeps() const;
 
   SnippetsStatus snippets_status_;
   SnippetsStatusChangeCallback snippets_status_change_callback_;
 
   bool require_signin_;
   SigninManagerBase* signin_manager_;
   PrefService* pref_service_;
 
   PrefChangeRegistrar pref_change_registrar_;
 
-  // The observer for the SigninManager.
-  ScopedObserver<SigninManagerBase, SigninManagerBase::Observer>
-      signin_observer_;
-
   DISALLOW_COPY_AND_ASSIGN(NTPSnippetsStatusService);
 };
 
 }  // namespace ntp_snippets
 
 #endif  // COMPONENTS_NTP_SNIPPETS_REMOTE_NTP_SNIPPETS_STATUS_SERVICE_H_