📰 Let the backend trigger sign in related refreshes

chrome/browser/ui/webui/snippets_internals_message_handler.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 "chrome/browser/ui/webui/snippets_internals_message_handler.h"
 
 #include <memory>
 #include <set>
 #include <utility>
 #include <vector>
@@ skipping 132 matching lines @@
   SendContentSuggestions();
 }
 
 void SnippetsInternalsMessageHandler::OnSuggestionInvalidated(
     const ntp_snippets::ContentSuggestion::ID& suggestion_id) {
   if (!dom_loaded_)
     return;
   SendContentSuggestions();
 }
 
+void SnippetsInternalsMessageHandler::OnFullRefreshRequired() {
+  if (!dom_loaded_) {

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

It might be worth moving this check into SendContentSuggestions().

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

There are a bunch of calls below that don't check that.

+    return;
+  }
+  SendContentSuggestions();
+}
+
 void SnippetsInternalsMessageHandler::ContentSuggestionsServiceShutdown() {}
 
 void SnippetsInternalsMessageHandler::HandleRefreshContent(
     const base::ListValue* args) {
   DCHECK_EQ(0u, args->GetSize());
 
   dom_loaded_ = true;
 
   for (std::pair<const Category, DismissedState>& category_state_pair :
        dismissed_state_) {
@@ skipping 233 matching lines @@
 
 void SnippetsInternalsMessageHandler::OnDismissedSuggestionsLoaded(
     Category category,
     std::vector<ContentSuggestion> dismissed_suggestions) {
   if (dismissed_state_[category] == DismissedState::HIDDEN)
     return;
   dismissed_suggestions_[category] = std::move(dismissed_suggestions);
   dismissed_state_[category] = DismissedState::VISIBLE;
   SendContentSuggestions();
 }