Change NTPSnippetsBridge to read from ContentSuggestionsService

chrome/browser/android/ntp/ntp_snippets_bridge.h

Index: chrome/browser/android/ntp/ntp_snippets_bridge.h
diff --git a/chrome/browser/android/ntp/ntp_snippets_bridge.h b/chrome/browser/android/ntp/ntp_snippets_bridge.h
index 11dc4dcec42e1cc0ef678c2a012c55e0a3d77487..05fb4e81db76d75dccf5bbc189e7a648af198f67 100644
--- a/chrome/browser/android/ntp/ntp_snippets_bridge.h
+++ b/chrome/browser/android/ntp/ntp_snippets_bridge.h
@@ -12,6 +12,9 @@
 #include "base/scoped_observer.h"
 #include "base/task/cancelable_task_tracker.h"
 #include "components/history/core/browser/history_service.h"
+#include "components/ntp_snippets/content_suggestions_category.h"
+#include "components/ntp_snippets/content_suggestions_category_status.h"
+#include "components/ntp_snippets/content_suggestions_service.h"
 #include "components/ntp_snippets/ntp_snippets_service.h"

[Marc Treib, 2016/07/20 09:17:38]

Not needed anymore

[Philipp Keck, 2016/07/20 13:20:02]

Moved to .cc file, it's needed there.

 
 namespace gfx {
@@ -20,7 +23,8 @@ class Image;
 
 // The C++ counterpart to SnippetsBridge.java. Enables Java code to access
 // the list of snippets to show on the NTP
-class NTPSnippetsBridge : public ntp_snippets::NTPSnippetsServiceObserver {
+class NTPSnippetsBridge
+    : public ntp_snippets::ContentSuggestionsService::Observer {
  public:
   NTPSnippetsBridge(JNIEnv* env,
                     const base::android::JavaParamRef<jobject>& j_profile);
@@ -46,9 +50,9 @@ class NTPSnippetsBridge : public ntp_snippets::NTPSnippetsServiceObserver {
                       const base::android::JavaParamRef<jobject>& callback,
                       const base::android::JavaParamRef<jstring>& jurl);
 
-  // Returns a reason why the snippet service is disabled, or 0 if it isn't.
-  // See NTPSnippetsService::DisabledReason for more info.
-  int GetDisabledReason(JNIEnv* env,
+  // Returns the status of the ARTICLES category.
+  // See ContentSuggestionsCategoryStatus for more info.
+  int GetCategoryStatus(JNIEnv* env,
                         const base::android::JavaParamRef<jobject>& obj);
 
   static bool Register(JNIEnv* env);
@@ -56,25 +60,26 @@ class NTPSnippetsBridge : public ntp_snippets::NTPSnippetsServiceObserver {
  private:
   ~NTPSnippetsBridge() override;
 
-  // NTPSnippetsServiceObserver overrides
-  void NTPSnippetsServiceLoaded() override;
-  void NTPSnippetsServiceShutdown() override;
-  void NTPSnippetsServiceDisabledReasonChanged(
-      ntp_snippets::DisabledReason disabled_reason) override;
+  // ContentSuggestionsService::Observer overrides
+  void OnNewSuggestions() override;
+  void OnCategoryStatusChanged(
+      ntp_snippets::ContentSuggestionsCategory category,
+      ntp_snippets::ContentSuggestionsCategoryStatus new_status) override;
+  void ContentSuggestionsServiceShutdown() override;
 
   void OnImageFetched(base::android::ScopedJavaGlobalRef<jobject> callback,
                       const std::string& snippet_id,
                       const gfx::Image& image);
 
-  ntp_snippets::NTPSnippetsService* ntp_snippets_service_;
+  ntp_snippets::ContentSuggestionsService* content_suggestions_service_;
   history::HistoryService* history_service_;
   base::CancelableTaskTracker tracker_;
 
   // Used to notify the Java side when new snippets have been fetched.
   base::android::ScopedJavaGlobalRef<jobject> observer_;
-  ScopedObserver<ntp_snippets::NTPSnippetsService,
-                 ntp_snippets::NTPSnippetsServiceObserver>
-      snippet_service_observer_;
+  ScopedObserver<ntp_snippets::ContentSuggestionsService,

[Marc Treib, 2016/07/20 09:17:38]

nit: I'd move this somewhere else, or at least put an empty line before it. As
it is, it looks like it's related to |observer_| and the comment above.

[Philipp Keck, 2016/07/20 13:20:02]

Done.

+                 ntp_snippets::ContentSuggestionsService::Observer>
+      content_suggestions_service_observer_;
 
   base::WeakPtrFactory<NTPSnippetsBridge> weak_ptr_factory_;