[Remote suggestions] Move all decisions to fetch to the scheduler

components/ntp_snippets/content_suggestions_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_CONTENT_SUGGESTIONS_SERVICE_H_
 #define COMPONENTS_NTP_SNIPPETS_CONTENT_SUGGESTIONS_SERVICE_H_
 
 #include <map>
 #include <memory>
 #include <set>
@@ skipping 194 matching lines @@
   // from such lists, making dismissed suggestions reappear (if the provider
   // supports it).
   void ClearDismissedSuggestionsForDebugging(Category category);
 
   // The reference to the RemoteSuggestionsProvider provider should
   // only be set by the factory and only used for debugging.
   // TODO(jkrcal) The way we deal with the circular dependency feels wrong.
   // Consider swapping the dependencies: first constructing all providers, then
   // constructing the service (passing the remote provider as arg), finally
   // registering the service as an observer of all providers?
+  // TODO(jkrcal) Move the getter into the scheduler interface (the setter is
+  // then not needed any more). crbug.com/695447
   void set_remote_suggestions_provider(
       RemoteSuggestionsProvider* remote_suggestions_provider) {
     remote_suggestions_provider_ = remote_suggestions_provider;
   }
   RemoteSuggestionsProvider* remote_suggestions_provider_for_debugging() {
     return remote_suggestions_provider_;
   }
 
   // The reference to RemoteSuggestionsScheduler should only be set by the
   // factory. The interface is suited for informing about external events that
@@ skipping 124 matching lines @@
 
   // Provides order for categories.
   std::unique_ptr<CategoryRanker> category_ranker_;
 
   DISALLOW_COPY_AND_ASSIGN(ContentSuggestionsService);
 };
 
 }  // namespace ntp_snippets
 
 #endif  // COMPONENTS_NTP_SNIPPETS_CONTENT_SUGGESTIONS_SERVICE_H_