📰 Let the backend trigger sign in related refreshes

components/ntp_snippets/content_suggestions_provider.h

Index: components/ntp_snippets/content_suggestions_provider.h
diff --git a/components/ntp_snippets/content_suggestions_provider.h b/components/ntp_snippets/content_suggestions_provider.h
index ddb6b468a884215c4f6ecf93aa5731a5403b2e9b..2efd0aef95292ee36e40677408af6955935d3acf 100644
--- a/components/ntp_snippets/content_suggestions_provider.h
+++ b/components/ntp_snippets/content_suggestions_provider.h
@@ -119,6 +119,12 @@ class ContentSuggestionsProvider {
   // from scratch.
   virtual void ClearCachedSuggestions(Category category) = 0;
 
+  // Called when the sign in state has changed. Should be used instead of
+  // directly registering with the SignInManager so that the
+  // ContentSuggestionService can control the order of the updates between
+  // the providers and the observers.
+  virtual void OnSignInStateChanged(){};

[Bernhard Bauer, 2016/11/21 15:48:15]

Nit: space before braces, and no semicolon at the end.

[Marc Treib, 2016/11/21 15:48:53]

nit: no ";". Also, space between "()" and "{}"?

Alternatively: Should we require all Providers to implement this?

[dgn, 2016/11/21 16:22:08]

Done.

AFAIK only the remote provider cares about the signin state, right? The others
just rely on whatever local thing backs them. So that would just add a dozen of
files to the changelog for not much. But if you feel strongly about it I'll make
the change.

[Marc Treib, 2016/11/21 16:52:30]

I guess it's conceivable that other providers might care, but at least so far
none of them does. So this is fine.

+
   // Used only for debugging purposes. Retrieves suggestions for the given
   // |category| that have previously been dismissed and are still stored in the
   // provider. If the provider doesn't store dismissed suggestions for the given